Skip to content

Integrated notifications: fading-effect bug with long list of notifications #19

Closed

Description

@Leleat

image

Activity

  1. Leleat commented on Nov 4, 2022

    @Leleat
    Author

    It does appear to be related to the scrollbar. Changing the scrollbar's width to 0 will expand the fade effect. Adding this.list.get_vscroll_bar().width = 0 to notifications.js will result in this

    image

    Downside is there is no scrollbar visible at all... although I think it actually looks good.

    Do you want me to open a PR with this? Or do you want to try to find a different/proper fix? Maybe you can do something in css since that's how you also enlarged the messages...

  2. self-assigned this
    on Nov 6, 2022
  3. pinned this issue on Nov 6, 2022
  4. qwreey commented on Nov 11, 2022

    @qwreey
    Owner

    I can't found that how to check when vscrollbar_visible of StScrollView will be change

    // ... basic
    list.vscroll.connect("hide")
    : not work, show too

    // Checking by height
    list.last_child.connect("notify::height")
    : not work

    // Checking by property notify
    list.connect("notify::vscrollbar_visible")
    : not work

    // Checking by child add/remove event
    list.last_child.first_child.first_child.connect("actor-added",()=>{Main.testt})
    : not work

    And also, i can't found any useful information in https://gjs-docs.gnome.org/st10~1.0_api/st.scrollview
    Did you know how to get event of 'vscrollbar_visible'?

  5. Leleat commented on Nov 11, 2022

    @Leleat
    Author

    I can't found that how to check when vscrollbar_visible of StScrollView will be change

    Sorry, don't know either :/. Maybe someone on the extension matrix channel may know...

  6. Leleat commented on Nov 23, 2022

    @Leleat
    Author

    Hmm, this sounds like the cause/fix of this issue... ?

    https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2190/

  7. added a commit that references this issue on Apr 29, 2023
    0717c9a
  8. qwreey commented on Apr 29, 2023

    @qwreey
    Owner

    image
    image

    This bug was fixed on dev branch. now it toggles overlay_scrollbars property automatically!

  9. unpinned this issue on May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions