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(Button): sbanken hover and error #2523

Merged
merged 7 commits into from
Aug 16, 2023
Merged

Conversation

snorrekim
Copy link
Contributor

@snorrekim snorrekim commented Jul 7, 2023

Summary

  • Updated the hover effect for sbanken buttons.
  • Updated sbanken error button styles and added screenshot tests for error states.

Missing error button hover, active, focus styles?

The new error button screenshots uncovered that these buttons do not have their own style for hover, active, focus in DNB/Eiendom theme. They just look the same as a regular button when interacted with. I don't know if that is intended or not, but might be a bug. (Sbanken theme has styles for these states).

@vercel
Copy link

vercel bot commented Jul 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eufemia ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 16, 2023 2:05pm

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 7, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit cccf829:

Sandbox Source
eufemia-starter Configuration

@langz langz changed the title feaet(Button): sbanken hover feat(Button): sbanken hover Jul 14, 2023
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from 483c1d0 to 27709af Compare July 31, 2023 08:45
@snorrekim snorrekim changed the title feat(Button): sbanken hover feat(Button): sbanken hover and error Aug 1, 2023
@snorrekim snorrekim marked this pull request as ready for review August 1, 2023 10:38
@snorrekim snorrekim requested a review from tujoworker August 8, 2023 10:51
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from 206f7b6 to 4de5c5c Compare August 10, 2023 10:13
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from fb3411e to 8d7def6 Compare August 10, 2023 14:22
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch 2 times, most recently from 4dfa98a to 71d12fe Compare August 11, 2023 11:15
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from 71d12fe to ed40653 Compare August 11, 2023 11:25
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch 2 times, most recently from 627b355 to 90e25d8 Compare August 11, 2023 12:45
@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from 90e25d8 to 8d7def6 Compare August 11, 2023 13:22
Copy link
Member

@tujoworker tujoworker left a comment

Choose a reason for hiding this comment

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

🔥 Great work, Snorre

@snorrekim snorrekim force-pushed the feat/button-sbanken-hover branch from 6192291 to cccf829 Compare August 16, 2023 14:01
@snorrekim snorrekim merged commit 3ea2850 into main Aug 16, 2023
@snorrekim snorrekim deleted the feat/button-sbanken-hover branch August 16, 2023 14:42
tujoworker pushed a commit that referenced this pull request Aug 17, 2023
## [10.6.0](v10.5.0...v10.6.0) (2023-08-17)

### 📝 Documentation

* add "Messageboxes" to FormStatus for the equivalent Figma name ([#2560](#2560)) ([bf6a19f](bf6a19f))
* add docs on how to link to a specific theme ([#2555](#2555)) ([54b9f1b](54b9f1b)), closes [#2374](#2374)
* add theming link to the contribution guide ([#2558](#2558)) ([e88bd72](e88bd72))

### ✨ Features

* **Anchor:** inline style sbanken ([#2550](#2550)) ([3511f09](3511f09))
* **Button:** sbanken hover and error ([#2523](#2523)) ([3ea2850](3ea2850))
* **Forms:** Extension (beta) for simplified implementation of web forms through tailored functionality for layout and data handling ([#2420](#2420)) ([74700c0](74700c0))
* **Input:** sbanken styling ([#2540](#2540)) ([db36f09](db36f09))
* **Textarea:** sbanken theme ([#2551](#2551)) ([d8cfb1c](d8cfb1c))

### 🐛 Bug Fixes

* **Forms:** fix circular imports issue ([#2566](#2566)) ([3ac4276](3ac4276))
* **NumberFormat:** accept options like maximumFractionDigits ([#2557](#2557)) ([2e09e80](2e09e80))
@tujoworker
Copy link
Member

🎉 This PR is included in version 10.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants