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

🐛 Bug Report: Placeholder Shown in the Allowed File Extensions Input #1018

Open
2 tasks done
ItzNotABug opened this issue Apr 8, 2024 · 4 comments
Open
2 tasks done
Labels
bug Something isn't working

Comments

@ItzNotABug
Copy link
Member

👟 Reproduction steps

  1. Go to your Project on Console
  2. Go to Storage > Settings Tab
  3. Scroll to Allowed file Extensions and Click on the extensions below to add them
  4. Notice that the placeholder text i.e. "Allowed file extensions (mp4, jpg, pdf, etc.)" moves to the right of the items added

👍 Expected behavior

Hide, remove the placeholder when the input is not empty.

👎 Actual Behavior

Placeholder is shown even when the input is not empty.

🎲 Appwrite version

Appwrite Cloud

💻 Operating system

Linux

🧱 Your Environment

Reproducible on Cloud & 1.5.x

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@ItzNotABug ItzNotABug added the bug Something isn't working label Apr 8, 2024
@sourabpramanik
Copy link
Contributor

That is just a helper text I don't see a bug in that, it is moreover an improvement rather than a bug imo

@ItzNotABug
Copy link
Member Author

@sourabpramanik A helper text or a placeholder is usually hidden when an input contains some form of text. This one keeps moving to the right which isn't something standard imo.

@sourabpramanik
Copy link
Contributor

sourabpramanik commented Apr 9, 2024

@sourabpramanik A helper text or a placeholder is usually hidden when an input contains some form of text.

Agreed for text type input fields this is not the right approach but for multi-select options like this one, this makes more sense.
Maybe we can add a sub-label along with the placeholder and remove the placeholder if the field is not empty

@FatumaA
Copy link
Contributor

FatumaA commented May 8, 2024

Can reproduce and would like to work on it if it is deemed a bug as above discussion

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

No branches or pull requests

3 participants