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

Activity UX/UI alignment #12

Merged
merged 2 commits into from
Jan 3, 2024
Merged

Activity UX/UI alignment #12

merged 2 commits into from
Jan 3, 2024

Conversation

kyle1morel
Copy link
Collaborator

Description

Aligns the activity view with the new UX/UI design in https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-3436.

Types of changes

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

Copy link

Coverage Report (Frontend)

Totals Coverage
Statements: 46.42% ( 175 / 377 )
Methods: 46.15% ( 36 / 78 )
Lines: 51.15% ( 111 / 217 )
Branches: 34.15% ( 28 / 82 )

@kyle1morel kyle1morel force-pushed the ux/activity-alignment branch 4 times, most recently from c9f0484 to 2584404 Compare December 28, 2023 23:13
Copy link

Coverage Report (Application)

Totals Coverage
Statements: 41.45% ( 63 / 152 )
Methods: 23.81% ( 5 / 21 )
Lines: 59.52% ( 50 / 84 )
Branches: 17.02% ( 8 / 47 )

@kyle1morel kyle1morel force-pushed the ux/activity-alignment branch 2 times, most recently from b2776d5 to 2be6e62 Compare December 29, 2023 22:07
Copy link
Member

@jujaga jujaga left a comment

Choose a reason for hiding this comment

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

Structurally looks good and seems to make sense. A few considerations to take a look at.

Copy link
Member

Choose a reason for hiding this comment

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

While we're still PoCing and changing this initial migration, I wonder if we should consider alphabetizing the columns, assuming there are no functional/business domain requirements for clustering the values together in the order they currently are.

app/src/db/models/submission.ts Show resolved Hide resolved
app/src/db/models/submission.ts Outdated Show resolved Hide resolved
app/src/services/chefs.ts Show resolved Hide resolved
frontend/src/components/form/Calendar.vue Show resolved Hide resolved
frontend/src/components/form/InputMask.vue Show resolved Hide resolved
frontend/src/components/form/InputNumber.vue Show resolved Hide resolved
frontend/src/components/form/InputText.vue Show resolved Hide resolved
frontend/src/components/form/Password.vue Show resolved Hide resolved
frontend/src/components/form/TextArea.vue Show resolved Hide resolved
@kyle1morel kyle1morel force-pushed the ux/activity-alignment branch from 2be6e62 to 68083e5 Compare January 3, 2024 19:37
@jujaga jujaga merged commit a08d642 into master Jan 3, 2024
19 of 20 checks passed
@jujaga jujaga deleted the ux/activity-alignment branch January 3, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants