Use thin sidebar resize handle

Bug #884068 reported by Cody Garver
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
BeatBox
Fix Released
Low
Victor Martinez
Lingo
Fix Released
Low
Unassigned
Marlin
Invalid
Undecided
Unassigned
elementary Stylesheet
Fix Released
Undecided
Unassigned

Bug Description

Since Marlin's left sidebar is resizable but doesn't use the big handle, BeatBox could do the same to maintain consistency and save a few px's.

Tags: bitesize
Changed in beat-box:
status: New → Confirmed
Revision history for this message
Scott Ringwelski (sgringwe) wrote :

Which CSS class?

Changed in beat-box:
importance: Undecided → Low
milestone: none → 0.3
tags: added: bitesize
Avi Romanoff (aroman)
Changed in lingo-dictionary:
status: New → Confirmed
importance: Undecided → Low
Changed in beat-box:
assignee: nobody → Victor Eduardo (victored)
status: Confirmed → In Progress
Revision history for this message
Victor Martinez (victored) wrote :

I fixed this by adding a new style class: sidebar-pane-separator. This needs to be added to egtk as well.

It should look like:

.sidebar-pane-separator {
    -GtkPaned-handle-size : 1;
    border-width : 0;
    background-image : none;
    background-color : shade (@bg_color, 0.75);
}

Changed in egtk:
status: New → Fix Committed
Changed in beat-box:
status: In Progress → Fix Released
Avi Romanoff (aroman)
Changed in lingo-dictionary:
status: Confirmed → Fix Committed
Revision history for this message
ammonkey (am-monkeyd) wrote :

In marlin we don't want all the pane and their separator from the column view to get the same style herited from the sidebar.
This why we target the specific class of the sidebar and it's pane separator like the following:

VarkaWidgetsHCollapsiblePaned {
    border-width: 0;
    -GtkPaned-handle-size: 1;
}

VarkaWidgetsHCollapsiblePaned.pane-separator,
VarkaWidgetsHCollapsiblePaned.pane-separator:prelight,
VarkaWidgetsHCollapsiblePaned.pane-separator:selected {
    background-image: -gtk-gradient (linear,
         left top,
         right top,
         from (shade (@bg_color, 1.30)),
         to (shade (@bg_color, 0.55)));
    background-color: alpha(#fff, 0.0);
}

Changed in marlin:
status: New → Invalid
Revision history for this message
Victor Martinez (victored) wrote :

ammonkey, thanks a lot for the information. That works great!

Changed in egtk:
milestone: none → 3.0
Avi Romanoff (aroman)
Changed in lingo-dictionary:
status: Fix Committed → Fix Released
Changed in egtk:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.