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

Warning of exceeding local storage quota gradually fills screen if editing is continued in 2.20.3 #8930

Closed
PeterLeGras opened this issue Feb 2, 2022 · 4 comments · Fixed by #8933
Labels
bug A bug - let's fix this!

Comments

@PeterLeGras
Copy link

URL

No response

How to reproduce the issue?

When the local storage quota is exceeded (for example by making a large changeset with many edits), the footer warning displays "You have made too many edits to back up. Consider saving your changes now". Since the 2.20.3 release, the warning message is appended to itself in the footer when additional edits are made (screenshot shows issue). This gradually fills the editing screen.

Screenshot(s) or anything else?

image

Which iD Editor versions do you see the issue on?

Released version at openstreetmap.org/edit

Which browsers are you seeing this problem on?

Chrome

@cicku
Copy link
Contributor

cicku commented Feb 2, 2022

Regression introduced by #8817

@tyrasd tyrasd added the bug A bug - let's fix this! label Feb 2, 2022
@1ec5
Copy link
Collaborator

1ec5 commented Feb 2, 2022

FYI, your browser may have a setting that increases the local storage quota. For example, in Mozilla-based browsers such as Firefox and SeaMonkey, set dom.storage.default_quota in about:config.

@cicku
Copy link
Contributor

cicku commented Feb 3, 2022

I'm more interested if there will be a hotfix released as 2.20.4. Why should users modify such settings?

@tyrasd
Copy link
Member

tyrasd commented Feb 3, 2022

Yes, there will be a hotfix v2.20.4 for this and #8928 released very soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants