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

3311 load more more prominent #3376

Merged
merged 5 commits into from
Feb 27, 2023

Conversation

Lakoja
Copy link
Collaborator

@Lakoja Lakoja commented Feb 23, 2023

Fixes #3311

I went for a gradient (instead of a zig-zag line) because that matches the other status separators better. And especially it integrates the existing separators on top and bottom of such a placeholder.

2b641fa65a85f8a8_kl

@connyduck
Copy link
Collaborator

Ok, if you don't want zig zag, then please do just the darker background, I think gradient is the worst of the options.

@Lakoja
Copy link
Collaborator Author

Lakoja commented Feb 25, 2023

Ok, if you don't want zig zag, then please do just the darker background, I think gradient is the worst of the options.

I want it but can't do it. :-)

So, ok, I'm using only another color now.

@connyduck connyduck merged commit 6b9db02 into tuskyapp:develop Feb 27, 2023
@Lakoja Lakoja deleted the 3311-load-more-more-prominent branch February 28, 2023 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the "load more" break stand out more
2 participants