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

💄 [#2311] Set feedback border-color #1158

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

jiromaykin
Copy link
Contributor

@jiromaykin jiromaykin commented Apr 15, 2024

After much discusion the decision is made to set the borders equally and use the info-colors.
Perhaps later a decision will be made to use different colors.

Latest updated issue: https://taiga.maykinmedia.nl/project/open-inwoner/issue/2427

Closed color-issue: https://taiga.maykinmedia.nl/project/open-inwoner/issue/2430

Design: https://www.figma.com/file/iKGhWhstaLIlFSaND2q7cE/OIP---Designs-(new)?node-id=2509%3A6766&mode=dev
old issue: https://taiga.maykinmedia.nl/project/open-inwoner/task/2311

@jiromaykin jiromaykin marked this pull request as ready for review April 15, 2024 15:24
@codecov-commenter
Copy link

codecov-commenter commented Apr 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.21%. Comparing base (b9e1ac8) to head (a2de593).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1158   +/-   ##
========================================
  Coverage    95.21%   95.21%           
========================================
  Files         1005     1005           
  Lines        37185    37185           
========================================
  Hits         35404    35404           
  Misses        1781     1781           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jiromaykin jiromaykin marked this pull request as draft April 16, 2024 12:45
@jiromaykin jiromaykin changed the title 💄 [#2311] Changed feedback border from blue to white 💄 [#2311] Change feedback border from blue to white Apr 18, 2024
@jiromaykin jiromaykin added the on hold Pause working for now, continued after decisions label Apr 25, 2024
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch from 0d6af07 to 05f4a07 Compare May 13, 2024 07:59
@jiromaykin jiromaykin changed the title 💄 [#2311] Change feedback border from blue to white 💄 [#2311] Set feedback border-color to primary May 13, 2024
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch from 489de34 to 6d11630 Compare May 13, 2024 08:14
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch from 6d11630 to b73042c Compare June 20, 2024 10:09
@jiromaykin jiromaykin changed the title 💄 [#2311] Set feedback border-color to primary 💄 [#2311] Set feedback border-color Jun 20, 2024
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch 3 times, most recently from 2570c42 to d013b9e Compare August 20, 2024 13:27
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch from d013b9e to 2d1b1a9 Compare August 27, 2024 08:04
@jiromaykin jiromaykin removed the on hold Pause working for now, continued after decisions label Aug 27, 2024
@jiromaykin jiromaykin force-pushed the feature/2311-feedback-button-border-white branch from 2d1b1a9 to 1dd0678 Compare August 27, 2024 08:12
@jiromaykin jiromaykin marked this pull request as ready for review August 27, 2024 10:29
@alextreme
Copy link
Member

🙈

@alextreme alextreme merged commit da110e7 into develop Aug 30, 2024
18 checks passed
@alextreme alextreme deleted the feature/2311-feedback-button-border-white branch August 30, 2024 09:53
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