Page MenuHomePhorge

Fix label of submit button in timezone conflict dialog
AbandonedPublic

Authored by waldyrious on Aug 29 2023, 21:32.

Details

Reviewers
None
Group Reviewers
O1: Blessed Committers
Summary

The timezone conflict dialog allows the user to either:

  • change the profile timezone setting to match the browser timezone
  • ignore the conflict

However, the submit button was explicitly labeled "Change Timezone", which is innacurate if the user picks the "Ignore Conflict" option from the dropdown menu.

This patch changes the label of the submit button to "Submit", so that it works for both ignoring the conflict and setting a new profile timezone.

Test Plan
  • Change your browser/system timezone to differ from your Phorge profile timezone
  • Click the notice that Phorge shows at the bottom left about the timezone mismatch
  • Confirm that the submit button now says "Submit" instead of "Change Timezone"

Diff Detail

Repository
rP Phorge
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

waldyrious edited the summary of this revision. (Show Details)

See D25420 for a conflicting change...

Thanks for the heads-up! That's a better approach indeed. I will abandon this patch in favor of that one.