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

Issues with UInput using type File #1524

Closed
TheAce4K opened this issue Mar 15, 2024 · 12 comments · Fixed by #1712
Closed

Issues with UInput using type File #1524

TheAce4K opened this issue Mar 15, 2024 · 12 comments · Fixed by #1712
Labels
bug Something isn't working

Comments

@TheAce4K
Copy link

Environment

  • Operating System: Darwin
  • Browser: Safari Version 16.6

Version

v2.14.2

Reproduction

https://ui.nuxt.com/components/input
If you go to the file type example you will see the issue

Description

Even when browsing the documentation I see this issue in my browser. The box is too small and the content is cropped. Please see image as it will be obvious:
image

Additional context

No response

Logs

No response

@TheAce4K TheAce4K added the bug Something isn't working label Mar 15, 2024
@CooperHash
Copy link

image

@HigherOrderLogic
Copy link
Contributor

Have you tried other browsers? The field seems fine on my side.

@moshetanzer
Copy link
Collaborator

@TheAce4K Yip this is a bug at least on safari. Will open a pr.

@CooperHash
Copy link

https://ui.nuxt.com/components/input

i can try to fix it

@moshetanzer
Copy link
Collaborator

Go for it @CooperHash

@CooperHash
Copy link

I'm sorry for it. This bug seems a little bit weird, and I can't figure out the fix method right now.
it seems safari's shadow content problem

@simoncdn
Copy link

Hello,
I've checked the input component with the type file for issue #1412 and this issue seems to come from the same thing: absolute position.

With absolute position:
withabsolute

With another position (here: block):
withblock

Copy link
Member

I tried but have no idea on how to fix that on Safari at the moment. The absolute positioning is required to achieve the desired result, otherwise you can just use a normal <input>.

@moshetanzer
Copy link
Collaborator

@benjamincanac maybe we should add a callout, or remove until we get it working?

Copy link
Member

Since I haven't found a solution to fix this, I decided to switch to a simpler design that will work everywhere:
CleanShot 2024-04-24 at 16.01.47@2x.png

What do you guys think?

@moshetanzer
Copy link
Collaborator

@benjamincanac Think light version text of "choose file" either should be made darker or lighter. This is how it looks IOS
image

Copy link
Member

benjamincanac commented Apr 25, 2024

Should we use the same color as the placeholder? I tried multiple shades and I think I'll keep it that way as it didn't look good.

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 a pull request may close this issue.

6 participants