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

UI bug with dots overflowing on the DashboardInputField #30

Closed
vmatsiiako opened this issue Nov 25, 2022 · 5 comments
Closed

UI bug with dots overflowing on the DashboardInputField #30

vmatsiiako opened this issue Nov 25, 2022 · 5 comments
Assignees
Labels
🐞 bug Something isn't working good first issue Good for newcomers help wanted Open for contributions from the community

Comments

@vmatsiiako
Copy link
Contributor

Bug Description

The dots that show up over a secret when it's not hovered, can sometimes go outside of the input field.
I tried a bunch of options but couldn't get the desired result easily with our current setup. I think the whole structure of how we generate those dots could be rethought. It's currently very lengthy and suboptimal. This is the file that would have to be changed.

To Reproduce

Steps to reproduce the behavior:

  1. Add a long key value
  2. Decrease your browser size if needed
  3. See error

Expected behavior

It should be the bahavior like "overflow-hidden" in tailwind

Screenshots

Screenshot 2022-11-25 at 11 06 20

@vmatsiiako vmatsiiako added 🐞 bug Something isn't working good first issue Good for newcomers help wanted Open for contributions from the community labels Nov 25, 2022
@rabbanirizwan
Copy link

@mv-turtle kindly assign me this

@vmatsiiako
Copy link
Contributor Author

Thank you so much @rabbanirizwan! This is a tricky one! If you want, I can explain to you how the current setup works. Let me know.

Feel free to join our community Slack if you have any questions or need any help. Here is the link: https://join.slack.com/t/infisical-users/shared_invite/zt-1kdbk07ro-RtoyEt_9E~fyzGo_xQYP6g

@vmatsiiako
Copy link
Contributor Author

I just fixed something else that is related to this. I think overall if made the structure of the file easier to understand.

@vmatsiiako
Copy link
Contributor Author

@rabbanirizwan after many tries, I think I just fixed this bug 🥳

Something that we want to do now is to make it possible to input multi-line environment variables. Is it something you would be interested in helping with? Let me know please :)

@vmatsiiako
Copy link
Contributor Author

@rabbanirizwan closing this issue, but please let me know if you want to contribute to anything. Any help is appreciated :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working good first issue Good for newcomers help wanted Open for contributions from the community
Projects
None yet
Development

No branches or pull requests

2 participants