Open
Bug 1026717
Opened 10 years ago
Updated 2 years ago
Options dialog implicitly cancelled by 'sync' links
Categories
(Firefox :: Settings UI, defect)
Tracking
()
UNCONFIRMED
People
(Reporter: bugzilla-mozilla, Unassigned)
Details
Steps to reproduce: Open the 'Options' dialog. Make some changes. Switch to 'Sync' tab. Decide you want to (create a sync account|sign in|learn more about older version) and click hyperlink. Actual results: All your changes will be discarded, the Options dialog is closed, and a new tab opens with your chosen destination. Expected results: The changes made in the Options dialog should not have been discarded without consulting me. Sure it's minor but it's also quite rude. Possible solutions, in decreasing preference: 1) Load target in background, without closing Options dialog 2) Ask user to apply or discard changes 3) Implicitly apply changes
Comment 1•10 years ago
|
||
Tim, do you know why these links close the options window? The ones from old sync open links but don't close the options dialog, for example...
Component: Untriaged → Preferences
Flags: needinfo?(ttaubert)
OS: Windows 7 → All
Hardware: x86 → All
Reporter | ||
Comment 2•10 years ago
|
||
This is non-issue for Firefox on Linux (well Ubuntu at least) because changes are implicitly saved. There is just Close, not OK/Cancel. But the dialog does close when clicking a link.
Comment 3•10 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #1) > Tim, do you know why these links close the options window? The ones from old > sync open links but don't close the options dialog, for example... I don't think that was a deliberate choice. Maybe we should just not close the dialog? On Windows where (iirc) the dialog is modal the user would have to close it themselves before being able to continue with the sync flow.
Flags: needinfo?(ttaubert)
Reporter | ||
Comment 4•9 years ago
|
||
Confirming still present in 34.0.5 on Windows platforms. (In reply to Tim Taubert [:ttaubert] from comment #3) > Maybe we should just not close the dialog? Either (a) this or (b) check for changed prefs and if found, prompt to save changes.
Updated•2 years ago
|
Severity: trivial → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•