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

content: input theme colors #3463

Closed
shawnnolanjr opened this issue Feb 20, 2024 · 5 comments · Fixed by #3471
Closed

content: input theme colors #3463

shawnnolanjr opened this issue Feb 20, 2024 · 5 comments · Fixed by #3471
Labels
triage New issues

Comments

@shawnnolanjr
Copy link

URL

https://ionicframework.com/docs/api/input#theming

Issue Description

Is this supposed to have different color schematics? It's working locally, but the docs example doesn't seem to be updating.

Screenshot 2024-02-20 at 12 22 13 PM
@shawnnolanjr shawnnolanjr added the triage New issues label Feb 20, 2024
@mapsandapps
Copy link
Contributor

Hi, it seems to be working in your screenshot. Could you be more specific about what's wrong? In ios mode, only the caret changes color (shown in yellow in your screenshot). In md mode, both the caret and underline will change color based on the color property.

@mapsandapps mapsandapps added the needs: reply Issues that need more information from the author label Feb 20, 2024
@ionitron-bot ionitron-bot bot removed the triage New issues label Feb 20, 2024
@shawnnolanjr
Copy link
Author

Hi, it seems to be working in your screenshot. Could you be more specific about what's wrong? In ios mode, only the caret changes color (shown in yellow in your screenshot). In md mode, both the caret and underline will change color based on the color property.

oh, ok. well it said it changes the color of the input, but i'm not seeing that change. But if this is working as expected, my bad.

@ionitron-bot ionitron-bot bot added triage New issues and removed needs: reply Issues that need more information from the author labels Feb 20, 2024
@averyjohnston
Copy link
Contributor

This isn't the first time I've seen someone get confused about this, so I'm going to put up a PR to clarify the docs 👀

@averyjohnston
Copy link
Contributor

I'm closing this as "not planned" because the demo is working as intended, but I've added some language in #3471 that should hopefully clarify things for everyone going forward. Thank you for the report!

@averyjohnston averyjohnston closed this as not planned Won't fix, can't repro, duplicate, stale Feb 26, 2024
@shawnnolanjr
Copy link
Author

I'm closing this as "not planned" because the demo is working as intended, but I've added some language in #3471 that should hopefully clarify things for everyone going forward. Thank you for the report!

cool thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage New issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants