notification: Limit max height and enable vscroll#13640
Conversation
anaximeno
changed the title
anaximeno
force-pushed
the
notification/set-max-min-height-and-enable-vscroll
branch
from
5a12fa0 to
8306b83
Compare
anaximeno
changed the title
|
I think this is good for a notification's initial display, but maybe the max height can be increased, or even based on some percentage of the monitor height. Once it's in the notification applet, however, it should revert to its full size and remove the individual scrolling altogether - I think the parent being scrollable there is enough, and having both is confusing (scroll works for the outer but not the inner). |
… disable when in the applet menu
…ent leaves Fix `St:ERROR:../src/st/st-scroll-bar.c:662:handle_button_press_event_cb: assertion failed: (!priv->grab_device) Bail out! St:ERROR:../src/st/st-scroll-bar.c:662:handle_button_press_event_cb: assertion failed: (!priv->grab_device)`, this can happen if events are lost when the scrollbar's container is hidden or removed while dragging and can trigger a full break of Cinnamon.
anaximeno
force-pushed
the
notification/set-max-min-height-and-enable-vscroll
branch
from
8306b83 to
ba1f02b
Compare
Done. |
2 participants
Avoids cases where notifications would grow beyond the monitor screen height.
Before
After