Structure GUI improvements
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 2.2k
- Forks
- 242
- Ø Merge
- 11 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
I'm working mainly off the Python version but have been wanting UI customizations and so have been making changes to the C++ side and rebuilding the wheel. One of the things I've had to implement so far is customization for how the Structure GUI adds structures to itself. Right now it adds structures by the typeName, which itself is a hardcoded string in the respective structure class and set as a static variable. It's used all over the place in that context so I ended up adding a defining a new var setting it all in the header for the structure class I wanted to customize the GUI for, then modifying the structureGUI build method to check for this new var and set it in a collapsing header by that instead of its typeName. In the end my solution is kind of hacky and relies on type casting. I would like to request a more official implementation of this feature as well as more customizability for UI elements, etc. as built in.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit der Build-Methode von Structure GUI und den relevanten Headern der Structure-Klassen, die in der Issue beschrieben sind; bestimmte Dateien oder Tests werden nicht genannt. Legen Sie den Umfang eines offiziellen Mechanismus zur Anpassung von Strukturen und der integrierten UI-Anpassung fest und bestätigen Sie anschließend, dass die resultierende API den aktuellen Workaround mit Type-Casting vermeidet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp, python
- Bereich
- frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100