Skip to content

fix(file-upload): disabled state colors (#DS-3824) #806

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

Merged
merged 1 commit into from
Jun 3, 2025
Merged

Conversation

NikGurev
Copy link
Contributor

@NikGurev NikGurev commented Jun 2, 2025

Summary

  • added dev example for all states of file-upload (for future e2e tests)
  • fixed icons colors

before

Screenshot 2025-06-02 at 22 21 02

after

Screenshot 2025-06-02 at 22 19 34

@NikGurev NikGurev self-assigned this Jun 2, 2025
@NikGurev NikGurev added the bug Something isn't working label Jun 2, 2025
Copy link

github-actions bot commented Jun 2, 2025

Visit the preview URL for this PR (updated for commit 236b6f7):

https://koobiq-next--prs-806-8badf9nz.web.app

(expires Sun, 08 Jun 2025 10:40:34 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@lskramarov
Copy link
Contributor

Неожиданно много изменений для фикса, @NikGurev здесь рефакторинг или еще что то ?

@NikGurev
Copy link
Contributor Author

NikGurev commented Jun 3, 2025

Неожиданно много изменений для фикса, @NikGurev здесь рефакторинг или еще что то ?

привел к единому виду стили стейтов + фикс иконок

@rmnturov
Copy link
Contributor

rmnturov commented Jun 3, 2025

если что в шапке списка текст серый надо сделать
https://www.figma.com/design/sHcbGrJoOZmFA0DsJQIer9/%F0%9F%9F%A5-koobiq-%C2%B7-components?node-id=16380-86421&t=m6bZSJ9fmgY8tCl8-4

@NikGurev NikGurev merged commit ba43fdc into main Jun 3, 2025
4 checks passed
@NikGurev NikGurev deleted the bugfix/DS-3824 branch June 3, 2025 11:04
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 this pull request may close these issues.

4 participants