In https://git.reviewboard.kde.org/r/121665 a customization of bookmark icon paths was added. Now the icon of bookmarks is saved along with the bookmarks file by specifying an absolute path to the icon image, even for the default case. There are a couple of downsides with this approach - if the installation directory is changed e.g. by the distribution, all user bookmark icons will break - if we change the path to the default bookmark icon or its name, all user bookmark icons will break - there is no support for icons loaded via qrc, which is important e.g. for diskless systems I'd suggest an improved version which - provides a couple of standard bookmark icon variants in a well-known qrc subfolder. E.g. qrc:/marble/bookmarks/bookmark_red.png, qrc:/marble/bookmarks/bookmark_yellow.png, ... - has a list view or similar that displays those icons in the dialog for easy selection by the user - optionally allows to insert further user-defined icons which could then use absolute paths to file system locations
I want to work on this feature and hence want clarification on a few things. 1. How should this new list of predefined icons be shown to the user, should I add a combo box below the icon path using which a user can select one of the predefined icons or should I add a button which on being clicked opens a new dialog showing the predefined icons. 2. Should I retain the existing way of adding a custom icon ( by clicking on the icon image to select an image file) or should I remove it and add "Custom" as one of the options of combo box. In case the icon selection is via a dedicated dialog window should I add a custom button in this dialog along with the list of all predefined icons. 3. When a user adds a custom icon image for a particular bookmark should this new icon be saved in the predefined icon list for future references or not?
Created attachment 96614 [details] Bookmark chooser dialog. I have submitted a patch for this here https://git.reviewboard.kde.org/r/126729/
Fixed in Applications/16.04 and later.