! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Can't eject media, button is hidden under scroll bar
Status:
RESOLVED: FIXED

Comments

Description Vinzenz Vietzke 2019-04-22 00:57:51 CEST
Created attachment 8439 
eject icon hidden under scroll bar

See screenshot attached.

What works for me as a solution somehow is to wait a second until the scrollbar is resized. Just moving the cursor to the center of the sidebar, at least far enough so the scrollbar loses the focus.

Nevertheless that's no real proper solution. Maybe the icon could force some more padding to the scrollbar initially?
Comment 1 ToZ editbugs 2019-04-22 02:46:01 CEST
This only happens with overlay scrollbars. This gtk3 snippet will create the space required to make the eject buttons reachable:

/* make eject buttons reachable */
.thunar .sidebar .view {
	padding-right: 10px;
}
Comment 2 Vinzenz Vietzke 2019-04-23 20:57:07 CEST
Thanks! But where should this snippet go to fix the problem permanently?
Pushing it to every Gtk theme can't be the way, right?
Comment 3 Andre Miranda editbugs 2019-04-24 02:45:16 CEST
Created attachment 8450 
alternative approach

The attached patch is an alternative approach which does not use CSS.
I also noticed that it takes at least 12px so the scrollbar won't overlap when the cursor is over it.
Another problem is that the first time you plug an external drive, the eject button is very close to the sidebar's border, but if you eject and plug it back the back will be a bit far. This behavior is already present but makes the extra padding even worse.
Comment 4 Vinzenz Vietzke 2019-05-09 11:45:22 CEST
Thanks, that looks much better now!
Could we get this patch merged?
Comment 5 Andre Miranda editbugs 2019-05-14 00:31:43 CEST
I'm not merging any because they solve this problem creating another: the button now is too far from the border which means less room for the device label.
Also AFAIK there is no (non-hacky) way to the sidebar width.
So I hoping for third approach.
Comment 6 Andre Miranda editbugs 2019-06-12 02:27:03 CEST
*** Bug 15593 has been marked as a duplicate of this bug. ***
Comment 7 Theo Linkspfeifer editbugs 2019-10-23 14:21:34 CEST
*** Bug 16078 has been marked as a duplicate of this bug. ***
Comment 8 Theo Linkspfeifer editbugs 2019-10-29 01:16:49 CET
Created attachment 9160 
diff

Try this.

The spinner does not seem to require any extra padding. For me it only appeared when unmounting a device, and thus always to the left of the eject symbol.
Comment 9 Theo Linkspfeifer editbugs 2019-10-29 14:32:26 CET
Having an extra column for the eject symbol reduces the available width for all other labels though.
Comment 10 Theo Linkspfeifer editbugs 2019-10-30 16:22:08 CET
Related report: Bug 15708
Comment 11 Andre Miranda editbugs 2019-11-06 05:28:08 CET
Created attachment 9187 
fix.diff

(In reply to Theo Linkspfeifer from comment #9)
> Having an extra column for the eject symbol reduces the available width for
> all other labels though.
Correct. And that's something I want to avoid.

See the attached diff, I was able to get rid of the eject_column (idk, maybe the problem it was meant to workaround was fixed in gtk) and made the padding conditional to the scroll bar visibility.
Comment 12 Git Bot editbugs 2019-12-16 16:36:09 CET
Andre Miranda referenced this bugreport in commit 78c15641e0aa6f780c6766056a848428ab1d8e60

Extra padding for Eject button when scrollbar is visible (Bug #15312)

https://git.xfce.org/xfce/thunar/commit?id=78c15641e0aa6f780c6766056a848428ab1d8e60
Comment 13 Andre Miranda editbugs 2019-12-16 16:37:50 CET
I've pushed to master, please let me know if it works well for you before closing this bug and cherry picking into xfce-4.14 branch.
Comment 14 Theo Linkspfeifer editbugs 2019-12-17 12:18:38 CET
Works well.
Comment 15 Git Bot editbugs 2019-12-19 02:43:21 CET
Andre Miranda referenced this bugreport in commit 31cf70d534afa7a9ea5ad75a024fa880ea4a4667

Extra padding for Eject button when scrollbar is visible (Bug #15312)

https://git.xfce.org/xfce/thunar/commit?id=31cf70d534afa7a9ea5ad75a024fa880ea4a4667

Bug #15312

Reported by:
Vinzenz Vietzke
Reported on: 2019-04-22
Last modified on: 2019-12-19
Duplicates (2):
  • 15593 Scrollbar covers eject icon
  • 16078 Side Pane scroll bar overlays eject buttons

People

Assignee:
Xfce Bug Triage
CC List:
5 users

Version

Attachments

eject icon hidden under scroll bar (21.56 KB, image/png)
2019-04-22 00:57 CEST , Vinzenz Vietzke
no flags
alternative approach (1.34 KB, patch)
2019-04-24 02:45 CEST , Andre Miranda
no flags
diff (1.31 KB, patch)
2019-10-29 01:16 CET , Theo Linkspfeifer
no flags
fix.diff (9.19 KB, patch)
2019-11-06 05:28 CET , Andre Miranda
no flags

Additional information