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

Styling adjustments #59

Merged
merged 11 commits into from
Mar 24, 2023
Merged

Styling adjustments #59

merged 11 commits into from
Mar 24, 2023

Conversation

loneil
Copy link
Contributor

@loneil loneil commented Mar 23, 2023

Description

Styling items from https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-3050
More to come in another PR later on, grouping these ones together.

  1. Change login prompt re grammar/bold – QA01-Log in

image

  1. Fix log in button styling – QA03-Log in button styling

image

  1. Fix navigation links to incorporate styling – QA06-Links

image

image

  1. Fix style of bucket header – QA05-Bucket hyperlink

image

  1. Fix style of bucket screen text – QA07-Text under select a bucket

image

  1. Recolour the action buttons – QA08-Recolour action buttons

image

image

image

  1. Fix colour styling of bucket icon – QA15-Bucket icon colour styling

image

  1. Fix colour styling of rows – QA16-Row highlight colour

image

image

  1. make loading bar at top of viewport fixed so page doesn't drop down when loading

Types of changes

New feature (non-breaking change which adds functionality)

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

@github-actions
Copy link

Coverage Report (Application)

Totals Coverage
Statements: 75% ( 51 / 68 )
Methods: 62.5% ( 5 / 8 )
Lines: 82.61% ( 38 / 46 )
Branches: 57.14% ( 8 / 14 )

@github-actions
Copy link

github-actions bot commented Mar 23, 2023

Coverage Report (Frontend)

Totals Coverage
Statements: 5.64% ( 44 / 780 )
Methods: 0% ( 0 / 181 )
Lines: 8.73% ( 42 / 481 )
Branches: 1.69% ( 2 / 118 )

@loneil loneil force-pushed the feature/uiQaFixes branch 3 times, most recently from b62767e to ac07826 Compare March 23, 2023 05:35
@loneil loneil marked this pull request as draft March 23, 2023 05:42
@loneil loneil force-pushed the feature/uiQaFixes branch 2 times, most recently from ff82cc5 to 3594376 Compare March 24, 2023 05:34
loneil added 7 commits March 23, 2023 22:36
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
@loneil loneil force-pushed the feature/uiQaFixes branch from 3594376 to 0f605bc Compare March 24, 2023 05:36
Signed-off-by: Lucas ONeil <[email protected]>
@loneil loneil force-pushed the feature/uiQaFixes branch from 0f605bc to 744a7f7 Compare March 24, 2023 05:51
@loneil loneil changed the title WIP Styling adjustments Mar 24, 2023
@loneil loneil marked this pull request as ready for review March 24, 2023 06:03
Comment on lines +62 to +64
// Primary color overrides for buttons and action items (checkboxes etc)
// Note this could be eventually replaced by a custom themeing (which has JUST been introduced in Primevue)
// once it is more settled implementation-wise
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wouldn't adopt the PrimeVue open-source theme customization yet, just came out and has all these notes about changes. This is easy enough at this point

loneil added 2 commits March 23, 2023 23:26
Signed-off-by: Lucas ONeil <[email protected]>
Signed-off-by: Lucas ONeil <[email protected]>
@kamorel kamorel merged commit 11da6d6 into master Mar 24, 2023
@kamorel kamorel deleted the feature/uiQaFixes branch March 24, 2023 20:20
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.

4 participants