Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(button): Remove extra space caused by input type="file"#471

Closed
andreadellacorte wants to merge 1 commit intonostalgic-css:developfrom
andreadellacorte:patch-1
Closed

fix(button): Remove extra space caused by input type="file"#471
andreadellacorte wants to merge 1 commit intonostalgic-css:developfrom
andreadellacorte:patch-1

Conversation

@andreadellacorte
Copy link

Description
Currently, elements have opacity: 0 rather than display: none - This is causing them to actually take space, causing an issue on mobile where the expands to the right.

image

This change fixes that.

Compatibility
No

Caveats
None

@trezy
Copy link
Member

trezy commented Nov 10, 2021

Thanks for the contribution! You'll need to update your commit message to comply with conventional commits before the checks will pass.

Also, did you verify that the file input still works on all devices after making this change? From what I recall, the input is set to opacity: 0 because it still needs to be clickable. Setting it to display: none; would theoretically leave us with a pretty but non-functional component.

@andreadellacorte andreadellacorte changed the title Remove extra space caused by input type="file" fix(button): Remove extra space caused by input type="file" Nov 10, 2021
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.

2 participants