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

browsersettting: fallback to localStorage/uiDefault for undefined setting #11382

Conversation

codewithvk
Copy link
Contributor

@codewithvk codewithvk commented Mar 20, 2025

browsersettting: fallback to localStorage/uiDefault for undefined setting

When a browser setting is not defined, use localStorage/uiDefault as a fallback. This fixes the regression related to the saveAsMode issue. bug: #11297

Change-Id: Ic035f3a5af95f903aff4934e998dd0a38675ccdc

@codewithvk codewithvk requested a review from Rash419 March 20, 2025 11:32
@codewithvk codewithvk force-pushed the private/codewithvk/allow_ui_default_settings branch 2 times, most recently from 7d9a71e to c501a6c Compare March 20, 2025 12:06
@codewithvk codewithvk changed the title browsersettting: use uiDefault fallback for missing setting browsersettting: fallback to localStorage/uiDefault for undefined setting Mar 20, 2025
@codewithvk codewithvk force-pushed the private/codewithvk/allow_ui_default_settings branch from c501a6c to f8e5637 Compare March 20, 2025 12:08
@codewithvk codewithvk force-pushed the private/codewithvk/allow_ui_default_settings branch from 7b02459 to 2025d7c Compare March 20, 2025 12:16
Copy link
Contributor

@Rash419 Rash419 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@codewithvk codewithvk force-pushed the private/codewithvk/allow_ui_default_settings branch from 2025d7c to e3a6cf2 Compare March 21, 2025 06:19
…ting

Signed-off-by: codewithvk <vivek.javiya@collabora.com>
Change-Id: Ic035f3a5af95f903aff4934e998dd0a38675ccdc
@codewithvk codewithvk force-pushed the private/codewithvk/allow_ui_default_settings branch from e3a6cf2 to 2e5e2c3 Compare March 21, 2025 08:39
@codewithvk codewithvk merged commit fe4fcd0 into CollaboraOnline:master Mar 22, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

SaveAsMode=group no longer working
2 participants