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

Polish CaseStatusSelectGroup dialog when changing status #4145

Merged
merged 4 commits into from
Dec 18, 2023

Conversation

wssheldon
Copy link
Contributor

Screenshot 2023-12-18 at 11 45 27 AM
Screenshot 2023-12-18 at 11 45 20 AM

@wssheldon wssheldon added enhancement New feature or request UI/UX labels Dec 18, 2023
<v-card-text>
This case was moved to the status
<v-chip :color="statusColors[selectedStatus]">{{ selectedStatus }}</v-chip> on
{{ selectedStatusTooltip }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you want to format this better and maybe add in the alt time zone tooltip?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

By formatting do you mean the code formatting or the format of the text on the page? The former is done for me automatically by the linter.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry I left off the important bit: format the time like with formatToUTC

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Rgr, done.

@wssheldon wssheldon merged commit 359d240 into master Dec 18, 2023
8 checks passed
@wssheldon wssheldon deleted the enhancement/polish-status-change-dialog branch December 18, 2023 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants