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

Mass action layout improved #6912

Merged
merged 3 commits into from Sep 4, 2023

Conversation

Simounet
Copy link
Member

@Simounet Simounet commented Sep 1, 2023

Some improvements for the main top parts of the homepage. More mobile friendly and less clicks for adding tags. Should fix #6818 by removing the click on the tag before being able to submit the form. I'm not sure if it's better with the icon before or after the input. Visually better with the icon before but I think the user flow is better with the icon after.

A. wallabag-icon-before
B. wallabag-icon-after

Before

wallabag-desktop-before

wallabag-mobile-before

After

wallabag-desktop-after

wallabag-mobile-after

@Simounet Simounet added the UI label Sep 1, 2023
@Simounet Simounet changed the title Feat/tag mass action improved Mass action layout improved Sep 1, 2023
@j0k3r j0k3r added this to the 2.6.6 milestone Sep 1, 2023
j0k3r
j0k3r previously approved these changes Sep 1, 2023
Copy link
Member

@j0k3r j0k3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As there is a title on the label (the one after the input), it seems ok to me!

@Simounet
Copy link
Member Author

Simounet commented Sep 1, 2023

I'm sorry but I lost your approval after updating the code for using the color variable.
Do you prefer the A or B version?

@j0k3r
Copy link
Member

j0k3r commented Sep 4, 2023

A :)

j0k3r
j0k3r previously approved these changes Sep 4, 2023
nicosomb
nicosomb previously approved these changes Sep 4, 2023
Copy link
Member

@nicosomb nicosomb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A

@Simounet Simounet dismissed stale reviews from nicosomb and j0k3r via 4b04cd5 September 4, 2023 10:00
@Simounet Simounet force-pushed the feat/tag-mass-action-improved branch from 36b134f to 4b04cd5 Compare September 4, 2023 10:00
@Simounet
Copy link
Member Author

Simounet commented Sep 4, 2023

I changed the code for the A version. You have to approve it again @j0k3r @nicosomb , sorry about that.

@j0k3r j0k3r merged commit 8ac80e9 into wallabag:2.6 Sep 4, 2023
19 checks passed
@Simounet Simounet deleted the feat/tag-mass-action-improved branch September 4, 2023 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants