Bug 434517

Summary: Remove redundant close button at the bottom of the preview window
Product: [Applications] ark Reporter: popov895 <popov895>
Component: generalAssignee: Elvis Angelaccio <elvis.angelaccio>
Status: RESOLVED FIXED    
Severity: wishlist CC: aacid, justin.zobel, nate, popov895, rthomsen6
Priority: NOR    
Version: 20.12.3   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=413434
https://bugs.kde.org/show_bug.cgi?id=434519
Latest Commit: Version Fixed In: 21.12
Attachments: Ark preview window

Description popov895 2021-03-16 23:26:45 UTC
Created attachment 136757 [details]
Ark preview window

SUMMARY

Preview windows contain a redundant close button that duplicates the close button in the title bar. Also, removing this button will free up additional vertical space.

SOFTWARE/OS VERSIONS

Linux/KDE Plasma: KDE neon 5.21 User Edition / X11
KDE Plasma Version: 5.21.2
KDE Frameworks Version: 5.80.0
Qt Version: 5.15.2
Comment 1 Justin Zobel 2021-03-17 21:31:32 UTC
Thanks for the suggestion, have moved to the wishlist section.
Comment 2 Bug Janitor Service 2021-10-01 14:55:43 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/ark/-/merge_requests/67
Comment 3 Bug Janitor Service 2021-10-01 14:55:45 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/ark/-/merge_requests/67
Comment 4 popov895 2021-10-01 16:36:23 UTC
There is already a MR for this issue https://invent.kde.org/utilities/ark/-/merge_requests/60 (two months old).
Comment 5 Nate Graham 2021-10-04 15:08:13 UTC
Git commit 5e9c140e1ef9f7a7cad61b3889cf7f820dccf4f4 by Nate Graham, on behalf of Eugene Popov.
Committed on 04/10/2021 at 15:08.
Pushed by ngraham into branch 'master'.

Remove redundant close button at the bottom of the preview window

This commit removes a redundant close button that duplicates the close
button in the title bar. Also, the preview window can be closed using
the keyboard shortcuts Ctrl+W / Alt+f4 / Esc.
FIXED-IN: 21.12

M  +4    -7    part/arkviewer.cpp
M  +1    -14   part/arkviewer.ui

https://invent.kde.org/utilities/ark/commit/5e9c140e1ef9f7a7cad61b3889cf7f820dccf4f4