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

set TagsInput webkit-calendar-picker icon to 0 opacity #2069

Merged
merged 3 commits into from
Apr 7, 2022
Merged

set TagsInput webkit-calendar-picker icon to 0 opacity #2069

merged 3 commits into from
Apr 7, 2022

Conversation

blite
Copy link
Contributor

@blite blite commented Mar 31, 2022

removes webkit-calendar-picker icon on tagsinput

before:
161101068-d8f95234-aea9-47d0-9523-3fc5471a6d53

after:
image

I did not like that I had to add another css file for this. Also don't love all the auto generated files in this commit.

@blite
Copy link
Contributor Author

blite commented Mar 31, 2022

This is per discussion at #2067

Maybe we should add back in a SVG icon, but for now just having that weird hard to debug icon out is IMO a good thing.

I sort of agree on the datalist thing which is why I created an issue before doing that. My only hesistation on the datalist is other unknown bugs, but we could get that with either solution.

@blite
Copy link
Contributor Author

blite commented Mar 31, 2022

For some reason display:none didn't seem to work, it would obviously be the ideal solution here.

@@ -42,7 +42,7 @@
x-on:blur="createTag()"
x-model="newTag"
{{ $getExtraInputAttributeBag()->class([
'block w-full border-0',
'tags-input block w-full border-0',
Copy link
Member

Choose a reason for hiding this comment

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

Lets stick with utility classes and rename this webkit-calendar-picker-indicator-opacity-0 maybe?

@@ -42,7 +42,7 @@
x-on:blur="createTag()"
x-model="newTag"
{{ $getExtraInputAttributeBag()->class([
'block w-full border-0',
'tags-input block w-full border-0',
Copy link
Member

Choose a reason for hiding this comment

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

Lets stick with utility classes and rename this webkit-calendar-picker-indicator-opacity-0 maybe?

@danharrin danharrin marked this pull request as draft April 3, 2022 12:21
@danharrin danharrin marked this pull request as draft April 3, 2022 12:21
@danharrin danharrin added bug Something isn't working ui labels Apr 3, 2022
@danharrin danharrin marked this pull request as ready for review April 7, 2022 14:21
@danharrin danharrin merged commit 2f3ffaf into filamentphp:2.x Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants