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.

Sign up for GitHub

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

Jump to bottom

Saving general settings successfully no longer scrolls to the success notification #18637

Closed
tsteur opened this issue Jan 17, 2022 · 0 comments · Fixed by #18638
Closed

Saving general settings successfully no longer scrolls to the success notification #18637

tsteur opened this issue Jan 17, 2022 · 0 comments · Fixed by #18638
Assignees
@peterhashair
Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone
4.7.0

Comments

@tsteur
Copy link
Member

tsteur commented Jan 17, 2022 •
edited

I checked with Matomo 4.6 and this seems to have worked there so it looks like this is a regression.

When saving settings, you now never really know whether it worked and it's confusing.
@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Jan 17, 2022
@tsteur tsteur added this to the 4.7.0 milestone Jan 17, 2022
@peterhashair peterhashair mentioned this issue Jan 17, 2022
Merged
11 tasks
@peterhashair peterhashair self-assigned this Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peterhashair peterhashair

Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Milestone
4.7.0
Development

Successfully merging a pull request may close this issue.

setting save success, scroll to notification matomo-org/matomo
2 participants
@tsteur @peterhashair