Bug 345919 - Bookmark icons use absolute paths in any case
Summary: Bookmark icons use absolute paths in any case
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Akshat Tandon
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2015-04-06 13:33 UTC by Dennis Nienhüser
Modified: 2016-04-02 06:24 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Bookmark chooser dialog. (58.25 KB, image/png)
2016-01-12 22:54 UTC, Akshat Tandon
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Nienhüser 2015-04-06 13:33:19 UTC
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
Comment 1 Akshat Tandon 2016-01-10 21:28:35 UTC
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?
Comment 2 Akshat Tandon 2016-01-12 22:54:55 UTC
Created attachment 96614 [details]
Bookmark chooser dialog.

I have submitted a patch for this here
https://git.reviewboard.kde.org/r/126729/
Comment 3 Dennis Nienhüser 2016-04-02 06:24:47 UTC
Fixed in Applications/16.04 and later.