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

Migrate to PrimeVue 4.0 #234

Merged
merged 41 commits into from
Jan 15, 2025
Merged

Migrate to PrimeVue 4.0 #234

merged 41 commits into from
Jan 15, 2025

Conversation

kyle1morel
Copy link
Collaborator

@kyle1morel kyle1morel commented Jan 7, 2025

Description

Update from PrimeVue 3.x to 4.x
Migrate from PrimeFlex to TailwindCSS + PrimeUI Plugin
Create custom PrimeVue theme override based on BCGov mandated colours
Update styling to as closely match previous version as possible.

Types of changes

Bug fix (non-breaking change which fixes an issue)

Documentation (non-breaking change with enhancements to documentation)
Breaking change (fix or feature that would cause existing functionality to change)

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

github-actions bot commented Jan 7, 2025

Coverage Report (Application)

Totals Coverage
Statements: 37.17% ( 1066 / 2868 )
Methods: 26.43% ( 129 / 488 )
Lines: 48.51% ( 701 / 1445 )
Branches: 25.24% ( 236 / 935 )

Copy link

github-actions bot commented Jan 7, 2025

Coverage Report (Frontend)

Totals Coverage
Statements: 46.18% ( 3224 / 6981 )
Methods: 35.21% ( 420 / 1193 )
Lines: 54.08% ( 1862 / 3443 )
Branches: 40.17% ( 942 / 2345 )

@naixin-zhangbc
Copy link

We are putting together a Figma file with screenshots and styling markups for all the needed changes. In the meantime, here are a couple bugs we noticed:

  • Drafts are not deleting after submission. If auto-save saved some drafts, after submission it should get rid of the drafts.
  • Once the button for activating a tooltip has been clicked, the tooltip message sticks on the screen across all pages. This happens to all tooltips.

@naixin-zhangbc
Copy link

Here are all the needed changes besides the bugs mentioned above. https://www.figma.com/design/udtqyDmqaYte03ANaS10S2/PAT?node-id=5716-457

wilwong89 and others added 26 commits January 13, 2025 14:04
Replacing depreciated component Dropdown with Select

Updated renamed components, inputswitch to toggleswitch, sidebar to drawer

Update Stepper component to new version

Update Accordion component to new format, fix submission intake navigation, fix map resize

Replace depreciated TabView component with Tabs

Primevue theming/CSS changes, handle input component blur events

WIP updating color vars
@kyle1morel kyle1morel force-pushed the feature/primevue-update branch from c98db7a to 38d2236 Compare January 13, 2025 22:23
@Subin1Doo
Copy link

Subin1Doo commented Jan 13, 2025

  • Under Housing tab, "Select the types of residential units being developed" section padding off

  • Under Housing tab, "No to all" button next to "Is this project associated with any of the following?" in the wrong spot, should be at far right. Looks like the 'No to all' button messed up the padding around the question as well.

  • data table row size is different, Drafts table bigger than General enquiries table. Can we match the drafts table with general enquiries table size.

@naixin-zhangbc
Copy link

It's looking good! A few more changes needed besides the ones from Subin. Please check out this new section in the Figma file:
https://www.figma.com/design/udtqyDmqaYte03ANaS10S2/PAT?node-id=5883-4540&t=p7O5cwzTEPoXpkPy-4

@naixin-zhangbc
Copy link

  • A new issue - the black icons in the image should be white.
Screenshot 2025-01-14 at 9 38 32 AM
  • The text in green alerts should use the BC Secondary (dark grey) type colour, e.g. the green banner for auto-save and for submission confirmation
Screenshot 2025-01-14 at 9 39 25 AM Screenshot 2025-01-14 at 9 56 28 AM

@kyle1morel kyle1morel marked this pull request as ready for review January 14, 2025 20:29
.vscode/settings.json Outdated Show resolved Hide resolved
frontend/src/assets/variables.scss Show resolved Hide resolved
Comment on lines 13 to 17
$app-link-text: #1a5a96;
$app-link-text-hover: #00f;
$app-hover: #4696EC;
$app-outline-on-primary: #fff;
$app-out-of-focus: #ccc;
Copy link
Contributor

@wilwong89 wilwong89 Jan 15, 2025

Choose a reason for hiding this comment

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

Consider putting all these hexcodes into the primitives section of ThemeOverride.ts and refer to them using var(). That way there's only one place to define extra hexcodes and using only one method to refer to them rather than mixing $app- and var() methods. Could get ride of variables.scss entirely that way.

Not sure if all these hexcodes are still being used.

Copy link
Collaborator Author

@kyle1morel kyle1morel Jan 15, 2025

Choose a reason for hiding this comment

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

@qhanson55 qhanson55 merged commit 12fe61e into master Jan 15, 2025
19 checks passed
@kyle1morel kyle1morel deleted the feature/primevue-update branch January 15, 2025 23:31
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.

6 participants