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

feat(navbar): align navbars to design system #688

Merged
merged 6 commits into from
Jun 28, 2021

Conversation

orbitalsqwib
Copy link
Contributor

Problem

Align navbars to the design system via custom components.

Closes #548

Solution

Features:

  • Implement custom styled tab components with theme styles

Improvements:

  • simplify layout
  • update styles to align with design system
  • minor updates to copy

Before & After Screenshots

BEFORE:
Screenshot 2021-06-11 at 3 45 13 PM
Screenshot 2021-06-11 at 3 45 23 PM
Screenshot 2021-06-11 at 3 45 04 PM
Screenshot 2021-06-11 at 3 44 55 PM

AFTER:
Screenshot 2021-06-11 at 3 44 17 PM
Screenshot 2021-06-11 at 3 44 24 PM
Screenshot 2021-06-11 at 3 44 03 PM
Screenshot 2021-06-11 at 4 16 20 PM

Tests

  • Check that design aligns to design system
  • Standard regression tests
    • Create a new checker from scratch
    • Create a new checker from template
    • Save draft and preview checker
    • Publish checker
    • Verify live checkers

@orbitalsqwib orbitalsqwib force-pushed the feat/design-system/navbar branch 2 times, most recently from b4551c1 to 9efa1ab Compare June 15, 2021 10:28
Base automatically changed from design-system to develop June 16, 2021 01:36
- implement common navbar components such as the navbar container,
  back-button and tabs
- implement theme styles for common navbar components aligned to the
  design system
- implement common navbar component styles for buttons
- use common navbar components
- simplify layout
- update styles to align with design system
- use common navbar components
- simplify layout
- update styles to align with design system
- use common navbar components
- simplify layout
- update styles to align with design system
- use common navbar components
- simplify layout
- update styles to align with design system
- extract styles to separate theme file
@lamkeewei lamkeewei changed the base branch from develop to design-system June 16, 2021 02:03
@lamkeewei lamkeewei changed the base branch from design-system to develop June 16, 2021 02:05
src/client/pages/FormBuilder.tsx Outdated Show resolved Hide resolved
src/client/pages/FormBuilder.tsx Outdated Show resolved Hide resolved
@orbitalsqwib
Copy link
Contributor Author

Note that this should also partially solve #393

Copy link
Contributor

@lamkeewei lamkeewei left a comment

Choose a reason for hiding this comment

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

lgtm!

@lamkeewei lamkeewei merged commit b843496 into develop Jun 28, 2021
@lamkeewei lamkeewei deleted the feat/design-system/navbar branch June 28, 2021 01:37
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.

[Design System] Navigation bars
2 participants