Skip to content
New issue

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

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

Already on GitHub? # to your account

frontend: settings conflict resolution is frustrating #1383

Open
1 of 7 tasks
ES-Alexander opened this issue Oct 3, 2024 · 0 comments · May be fixed by #1684
Open
1 of 7 tasks

frontend: settings conflict resolution is frustrating #1383

ES-Alexander opened this issue Oct 3, 2024 · 0 comments · May be fixed by #1684
Assignees
Labels
enhancement New feature or request ui UI related issues ux Has to do with the user experience

Comments

@ES-Alexander
Copy link
Contributor

Current behaviour

When there are settings differences between those stored locally on Cockpit and those backed up on the vehicle (in BlueOS), the current conflict resolution mechanism provides limited information about what is actually different (which is confusing), does not clearly identify what the different options mean (so decisions are unintuitive), and requires confirming each setting individually (which can be rather annoying).

Screenshot 2024-10-03 at 3 11 42 PM

Problem identified by Phil, and agreed with by @vshie and me.

Expected or desired behaviour

A better approach would be to:

  • Describe more clearly what it means to have a settings conflict
    • "Your local Cockpit configuration does not match the backup on the connected vehicle. Which values should be used?"
    • A help icon tooltip or an expandable section could clarify "Your [Cockpit instance / vehicle] has been used with a different [vehicle / Cockpit instance], and the {user} user settings were reconfigured."
      • If Cockpit has more recent settings then it has been used with a different vehicle, and vice versa
  • Allow confirming all conflicting settings at once
  • Provide descriptions/tooltips to explain what each setting is actually used for
  • Specify which set of settings is newer, and ideally how old they are
    • Storing a timestamp when they're updated would allow this to be done intuitively
    • By default, select the newer settings
  • Provide some form of semantic diff between the options (i.e. what is new, what is removed)
    • Ideally this would be able to show visual differences for Views and the like, but that's probably excessive
    • A simpler (but still much better than current) approach would be to just show a diff between the json for that setting (over multiple lines if necessary)
      • This could potentially skip "hash" rows, and others determined to be unhelpful for the differentiation
  • Provide reasonable "don't ask me again" options
    • Users using the "user" options as intended should be able to decide on a personal preference for this and not need to think about it all the time
image

Prerequisites

  • I have checked to make sure that a similar request has not already been filed or fixed.
@ES-Alexander ES-Alexander added enhancement New feature or request ui UI related issues ux Has to do with the user experience labels Oct 3, 2024
@ES-Alexander ES-Alexander changed the title frontend: settings conflict resolution needs more context frontend: settings conflict resolution is frustrating Oct 3, 2024
@rafaellehmkuhl rafaellehmkuhl self-assigned this Feb 13, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request ui UI related issues ux Has to do with the user experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants