Skip to content

Styling fixes and disabled state#56

Merged
janmeier merged 3 commits into
masterfrom
styling-fixes-and-disabled-state
Apr 27, 2026
Merged

Styling fixes and disabled state#56
janmeier merged 3 commits into
masterfrom
styling-fixes-and-disabled-state

Conversation

@janmeier

@janmeier janmeier commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Cherry-picking some commits from #53, to make it easier to review.

This PR contains no direct PAR related changes. It:

  • Clean ups how we do eID colors
  • Adds disabled state (to be used by PAR)
  • Adds loading state, and streamlines the spinner (to be used by PAR)

@netlify

netlify Bot commented Apr 27, 2026

Copy link
Copy Markdown

Deploy Preview for criipto-verify-react-storybook ready!

Name Link
🔨 Latest commit e24a449
🔍 Latest deploy log https://app.netlify.com/projects/criipto-verify-react-storybook/deploys/69ef1e85cf0d0f0008002d78
😎 Deploy Preview https://deploy-preview-56--criipto-verify-react-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Instead of hardcoding each hover color, we use HSL, and modify the lightness.
When we enable PAR, there is a short loading step before anything happens, while we initialize the request. While that happens, we need to be able to disable the button, so the user doesn't press multiple times.
@janmeier janmeier force-pushed the styling-fixes-and-disabled-state branch from 17364b2 to 01e88dd Compare April 27, 2026 08:25
…ttonComponent

We already had some loader styling, but it was only used in norwegian bankid. Now we are adding a component that can be used across all places.
@janmeier janmeier force-pushed the styling-fixes-and-disabled-state branch from 01e88dd to e24a449 Compare April 27, 2026 08:29
@janmeier janmeier marked this pull request as ready for review April 27, 2026 08:32
@janmeier janmeier requested review from jlndk and mickhansen April 27, 2026 08:32
@janmeier janmeier merged commit 65d41f1 into master Apr 27, 2026
6 checks passed
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.

2 participants