nuxt / ui

A UI Library for Modern Web Apps, powered by Vue & Tailwind CSS.
https://ui.nuxt.com
MIT License
3.45k stars 383 forks source link

Issues with UInput using type File #1524

Closed TheAce4K closed 1 month ago

TheAce4K commented 2 months ago

Environment

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

CooperHash commented 2 months ago

image

HigherOrderLogic commented 2 months ago

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

moshetanzer commented 2 months ago

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

CooperHash commented 2 months ago

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

i can try to fix it

moshetanzer commented 2 months ago

Go for it @CooperHash

CooperHash commented 2 months ago

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 commented 2 months ago

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
benjamincanac commented 2 months ago

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 commented 2 months ago

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

benjamincanac commented 1 month ago

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 commented 1 month ago

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

benjamincanac commented 1 month ago

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.