Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It will go to top on taht page when you click togle switch when it is open detail #220

Open
oyajun opened this issue Jul 1, 2022 · 2 comments · May be fixed by #773
Open

It will go to top on taht page when you click togle switch when it is open detail #220

oyajun opened this issue Jul 1, 2022 · 2 comments · May be fixed by #773
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@oyajun
Copy link
Contributor

oyajun commented Jul 1, 2022

top.webm
@mjakeman mjakeman added bug Something isn't working help wanted Extra attention is needed labels Aug 1, 2022
@oscfdezdz
Copy link
Collaborator

Related to

  • Toggle extensions without recreating the group

in #485

oscfdezdz added a commit that referenced this issue Feb 2, 2025
The items-changed signal is now emitted unchanged in the model from
ExmManager and is connected after binding the manager in
ExmInstalledPage from where the callback saves, emits items-changed and
restores focus.

This approach allows to keep binding GtkListBox to the extensions model
and not having to manage the changes manually.

Fix #220
oscfdezdz added a commit that referenced this issue Feb 2, 2025
The items-changed signal is now emitted unchanged in the model from
ExmManager and is connected after binding the manager in
ExmInstalledPage from where the callback saves, emits items-changed and
restores focus.

This approach allows to keep binding GtkListBox to the extensions model
and not having to manage the changes manually.

Fix #220
oscfdezdz added a commit that referenced this issue Feb 2, 2025
The items-changed signal is now emitted without adding or removing items
to the model from ExmManager and is connected after binding the manager
in ExmInstalledPage from where the callback saves, emits items-changed
adding and removing as done before in ExmManager, and restores focus.

This approach allows us to keep binding GtkListBox to the extensions
model and not having to manage the changes manually.

Fix #220
@oscfdezdz
Copy link
Collaborator

oscfdezdz commented Feb 2, 2025

#773 saves the scroll position scrolls when the sort enabled first option is active with any change in the switches and consequent change of focus but it doesn't prevent the expanders from collapsing and altering the position, that would be fixed when we get rid of the expander.

oscfdezdz added a commit that referenced this issue Feb 2, 2025
The items-changed signal is now emitted without adding or removing items
to the model from ExmManager and is connected after binding the manager
in ExmInstalledPage from where the callback saves, emits items-changed
adding and removing as done before in ExmManager, and restores focus.

This approach allows us to keep binding GtkListBox to the extensions
model and not having to manage the changes manually.

Fix #220
oscfdezdz added a commit that referenced this issue Feb 3, 2025
The items-changed signal is now emitted without adding or removing items
to the model from ExmManager and is connected after binding the manager
in ExmInstalledPage from where the callback saves, emits items-changed
adding and removing as done before in ExmManager, and restores focus.

This approach allows us to keep binding GtkListBox to the extensions
model and not having to manage the changes manually.

Fix #220
oscfdezdz added a commit that referenced this issue Feb 3, 2025
The items-changed signal is now emitted without adding or removing items
to the model from ExmManager and is connected after binding the manager
in ExmInstalledPage from where the callback saves, emits items-changed
adding and removing as done before in ExmManager, and restores focus.

This approach allows us to keep binding GtkListBox to the extensions
model and not having to manage the changes manually.

Fix #220
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants