Bug 450850 - Two "Go Full Screen" buttons in viewer page
Summary: Two "Go Full Screen" buttons in viewer page
Status: RESOLVED FIXED
Alias: None
Product: peruse
Classification: Applications
Component: general (other bugs)
Version First Reported In: master
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Remko van Wagensveld
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-25 22:07 UTC by Nate Graham
Modified: 2022-07-18 16:11 UTC (History)
1 user (show)

See Also:
Latest Commit: https://invent.kde.org/graphics/peruse/commit/f98ad615de11e6a847379a72c17182c916e6dde5
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Two "Go Full Screen" buttons (65.28 KB, image/png)
2022-02-25 22:07 UTC, Nate Graham 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2022-02-25 22:07:23 UTC 
Created attachment 147123 [details]
Two "Go Full Screen" buttons

See attached screenshot.
Comment 1 Bug Janitor Service 2022-07-02 15:20:25 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/graphics/peruse/-/merge_requests/30
Comment 2 Remko van Wagensveld 2022-07-18 16:11:26 UTC 
Git commit f98ad615de11e6a847379a72c17182c916e6dde5 by Remko van Wagensveld.
Committed on 16/07/2022 at 15:32.
Pushed by ngraham into branch 'master'.

Remove redundant 'Enter Fullscreen' Button

The mobile and desktop Actions already contain an 'Enter Fullscreen' Action.
These need to be kept, as the desktop Action has the functionality for keyboard
shortcuts ('f' and 'Esc') implemented.

M  +0    -7    src/app/qml/Book.qml

https://invent.kde.org/graphics/peruse/commit/f98ad615de11e6a847379a72c17182c916e6dde5