Bug 442863 - Bookmark icon overlaid on top of thumbnails needs visual polish
Summary: Bookmark icon overlaid on top of thumbnails needs visual polish
Status: CONFIRMED
Alias: None
Product: okular
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-23 20:49 UTC by Nate Graham
Modified: 2021-09-25 18:39 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Bookmark icons (799.93 KB, image/png)
2021-09-23 20:49 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2021-09-23 20:49:47 UTC
Created attachment 141848 [details]
Bookmark icons

I'm seeing this with current git master; look at the attached screenshot.
Comment 1 Nate Graham 2021-09-23 20:51:49 UTC
Bisecting now...
Comment 2 Nate Graham 2021-09-23 21:01:04 UTC
Hmm, maybe not a recent regression; git bisect shows that this has been the case for ages... have I really just been blind to it?
Comment 3 Laura David Hurka 2021-09-24 09:06:15 UTC
I think I have seen that before. But same here, I wasn’t aware of it. I never use bookmarks.
Comment 4 Albert Astals Cid 2021-09-24 09:46:17 UTC
Yes, if the page is bookmarked we show a bookmark on the page.

How is that a bug?
Comment 5 Nate Graham 2021-09-24 13:19:01 UTC
Oh I see, it's a feature. It isn't very visually polished so I thought it was a bug. :) Typically for a badge/overlay like this, we use an icon designed to look good in an overlay role called an emblem, of which there are many in the Breeze theme. And we usually put it in the corner, not right in the middle. I'll work on a merge request to improve the presentation.
Comment 6 Laura David Hurka 2021-09-24 17:27:26 UTC
Here it appears in the top-right corner. Looks a bit like a paper clip. Maybe it is not correctly positioned on hiDPI?

I think it doesn’t look very beautiful. And it is not very noticeable, when navigating through the document the thumbnails panel scrolls too fast. There is not only this emblem appearing, the whole viewport changes.

I think bookmarks should be visible in the main view. Like a speech bubble outside the left or right page border.
Comment 7 Albert Astals Cid 2021-09-25 16:44:30 UTC
It looked much better when bookmark was a solid yellow star, then it was not a star anymore, and thus it was less noticeable, the problem of Okular not being in control of the icons used in the app, it can't always make it look nice for all styles and icon set variations.

David is right though, the x positioning of the icon on your screenshot looks wrong.

I disagree with showing this information in the main view, once you start doing that you'll get reports of "shows the pdf wrong, some artifact in this location".
Comment 8 Nate Graham 2021-09-25 18:39:58 UTC
I agree, it should probably stay on the thumbnail view. The positioning does look like a hidpi positioning bug.

As for the icon chosen, we'll need to hardcode something in the app or pick an icon from the icon theme that is designed for this purpose--those would be emblems. Unfortunately there is no bookmark emblem in the Breeze theme, and I assume not in other themes either. I can create one and make Okular use it, and then hopefully other themes will fall back to Breeze and get the Breeze icon.