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.

Sign up for GitHub

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

Jump to bottom

[UI] Move to folder - Change color hover / defaut #195

Closed
TrucNguyenx opened this issue Feb 8, 2023 · 7 comments · Fixed by #222
Closed

[UI] Move to folder - Change color hover / defaut #195

TrucNguyenx opened this issue Feb 8, 2023 · 7 comments · Fixed by #222
Assignees
@trangiahuy031099 @huynhngocgiau

Comments

@TrucNguyenx
Copy link

TrucNguyenx commented Feb 8, 2023 •
edited
Loading

Improve UI - 'Move to folder'

  • Currently:
    Image

  • Design:

Image

Link: https://www.figma.com/proto/m0VZCJPKKj8uGOfZReXMYR/AesirX-Platform?page-id=552%3A1816&node-id=4681%3A25882&viewport=-8808%2C2780%2C1&scaling=min-zoom

cc @huynhngocgiau
@TrucNguyenx TrucNguyenx converted this from a draft issue Feb 8, 2023
@huynhngocgiau huynhngocgiau mentioned this issue Feb 17, 2023
Closed
@trangiahuy031099 trangiahuy031099 moved this to In Progress in AesirX Development Feb 17, 2023
trangiahuy031099 added a commit to trangiahuy031099/dam-app that referenced this issue Feb 20, 2023
@trangiahuy031099
aesirxio#195: move to folder
803f468
@trangiahuy031099 trangiahuy031099 mentioned this issue Feb 20, 2023
Merged
@trangiahuy031099 trangiahuy031099 moved this from In Progress to Awaiting QA in AesirX Development Feb 20, 2023
@lunguyenhat lunguyenhat moved this from Awaiting QA to Awaiting Review in AesirX Development Feb 20, 2023
@huynhngocgiau
Copy link

conflict a @kakahuy113

@trangiahuy031099 trangiahuy031099 moved this from Awaiting Review to Awaiting QA in AesirX Development Feb 21, 2023
@huynhngocgiau
Copy link

Please check again a @kakahuy113

  1. Add icon "Back"
  2. Delete "Cancel", "Move to folder" button
    image

@huynhngocgiau huynhngocgiau moved this from Awaiting QA to Todo in AesirX Development Feb 21, 2023
@trangiahuy031099
Copy link
Contributor

icon back can not be in root folder e and if dont have cancel or movetofolder button what the view do e ? c @TrucNguyenx just demo the hover and active and keep the same from what we get please ask truc for detail e

@huynhngocgiau
Copy link

Location for icon and text @kakahuy113
image

@trangiahuy031099 trangiahuy031099 moved this from Todo to In Progress in AesirX Development Feb 21, 2023
trangiahuy031099 added a commit to trangiahuy031099/dam-app that referenced this issue Feb 21, 2023
@trangiahuy031099
aesirxio#195: move to folder
ae0d659
@trangiahuy031099 trangiahuy031099 moved this from In Progress to Awaiting QA in AesirX Development Feb 21, 2023
@huynhngocgiau
Copy link

has updated
image

@huynhngocgiau huynhngocgiau moved this from Awaiting QA to Awaiting Merge in AesirX Development Feb 21, 2023
@lunguyenhat lunguyenhat moved this from Awaiting Merge to Awaiting Review in AesirX Development Feb 21, 2023
@lunguyenhat lunguyenhat moved this from Awaiting Review to Awaiting QA in AesirX Development Feb 21, 2023
@VanTrieuThanh
Copy link

VanTrieuThanh commented Feb 22, 2023 •
edited
Loading

Has update on the docker-qa file
Screenshot from 2023-02-22 09-56-36
But I see when click on the folder/file it has show the background
image

@VanTrieuThanh VanTrieuThanh moved this from Awaiting QA to Awaiting Merge in AesirX Development Feb 22, 2023
@VanTrieuThanh VanTrieuThanh moved this from Awaiting Merge to Todo in AesirX Development Feb 22, 2023
@trangiahuy031099 trangiahuy031099 moved this from Todo to Awaiting QA in AesirX Development Feb 22, 2023
@huynhngocgiau
Copy link

has updated
image

@huynhngocgiau huynhngocgiau moved this from Awaiting QA to Awaiting Merge in AesirX Development Feb 22, 2023
lunguyenhat added a commit that referenced this issue Feb 22, 2023
@lunguyenhat
Merge pull request #222 from kakahuy113/195
ad4e7e2 
#195: move to folder
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done in AesirX Development Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trangiahuy031099 trangiahuy031099

@huynhngocgiau huynhngocgiau

Labels
None yet
Projects
AesirX Development
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#195: move to folder trangiahuy031099/dam-app
4 participants
@VanTrieuThanh @trangiahuy031099 @huynhngocgiau @TrucNguyenx