Code review comment for lp:~daggerstab/stellarium/satellites-import-gui

Revision history for this message
Bogdan Marinov (daggerstab) wrote :

1. Fixed.

2. Because of 3., the QListWidget was replaced with a QListView, which has its own default style. Have a look at it.

3. Done.

4. A possible mechanism:
- add a "new" flag to the Satellite object that is not saved to JSON;
- add "new" to Satellites::Visibility and use getSatellites() to get all with that flag;
- add a "[newly added]" group to the drop-down list - it will display the satellites added since Stellarium has been started.

« Back to merge proposal