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

Update placeholder text colour #3062

Merged
merged 4 commits into from
Mar 1, 2024
Merged

Conversation

rmccar
Copy link
Contributor

@rmccar rmccar commented Feb 29, 2024

What is the context of this PR?

Fixes: #3043

Updates the placeholder text colour to differentiate it from the colour of the entered text. This was done by adding a new --ons-color-text-placeholder css variable and to differentiate renaming the --ons-color-placeholder variable to --ons-color-image-placeholder.

Also updates some code formatting.

How to review this PR

  • Spin up the DS and see that the placeholder text in the component search box is now grey and compare that against main.
  • Also do this with the input placeholder example

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@rmccar rmccar self-assigned this Feb 29, 2024
Copy link

netlify bot commented Feb 29, 2024

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 0a0e49c
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/65e04f90d4d1a6000896e9f5
😎 Deploy Preview https://deploy-preview-3062--ons-design-system-preview.netlify.app/patterns/guide/example-guide-overview
📱 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 site configuration.

@rmccar rmccar added the Bug Something isn't working label Feb 29, 2024
@rmccar rmccar merged commit cc6c097 into main Mar 1, 2024
9 checks passed
@rmccar rmccar deleted the bug/3043/update-placeholder-text-colour branch March 1, 2024 14:11
rmccar added a commit that referenced this pull request Mar 6, 2024
alessioventuriniAND added a commit that referenced this pull request Apr 4, 2024
…wcag 22 level aaa (#3119)

* ⬆️  Back To Top Link  ⬆️  (#2990)

* Page template for length

* Add macro

* Add component foundations

* Build stylesheet

* Add additional functionality

* Improved functionality

* Fix border and layout

* Fix resize behaviour

* Changes to CSS

* Add more tests

* Update VR Tests

* Change to full width

---------

Co-authored-by: Alessio Venturini <112873190+alessioventuriniAND@users.noreply.github.com>

* Description list not structured properly for screen readers (#3053)

* initial change

* update test

* update test

* update comment

* update comment

* Update placeholder text colour (#3062)

* Fix/3004/icon in footer not clickable (#3036)

* Fix checkbox appearance on safari (#3066)

* Adding the change in readme (#3067)

* updated youtube video to a more accessible one

* updated reference files

---------

Co-authored-by: Aditya Unnithan_ONS <56112669+adi-unni@users.noreply.github.com>
Co-authored-by: Precious Onyenaucheya <86783201+precious-onyenaucheya-ons@users.noreply.github.com>
Co-authored-by: rmccar <42928680+rmccar@users.noreply.github.com>
Co-authored-by: Bali Birch-Lee_ONS <149602681+balibirchlee@users.noreply.github.com>
Co-authored-by: SriHV <123635670+SriHV@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate text colour for input into accessiblePlaceholder
3 participants