Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

Add a subscribe button to empty feed #121

Merged
merged 7 commits into from
Mar 27, 2023
Merged

Conversation

Schmiddiii
Copy link
Collaborator

Licensing

  • I confirm that this is either my code or was released under the terms of a GPLv3-or-later compatible license. Also I agree to release it here under the terms of the GPLv3-or-later.

Description

This adds a "subscribe"-button to an empty feed page.

tubefeeder_subscribe_button

Ping @TheEvilSkeleton if anything can be improved in this regard.

Linked Issue

Related to #95.

@Schmiddiii Schmiddiii force-pushed the feed-subscribe-button branch from 45107f9 to e98f5ad Compare March 24, 2023 13:02
@Schmiddiii Schmiddiii changed the title 💄 Add a subscribe button to empty feed Add a subscribe button to empty feed Mar 24, 2023
Copy link
Contributor

@TheEvilSkeleton TheEvilSkeleton left a comment

Choose a reason for hiding this comment

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

My suggestions aren't final, but I think it's a good start. I'll test once the suggestions are added

data/resources/ui/feed_page.ui Outdated Show resolved Hide resolved
data/resources/ui/feed_page.ui Show resolved Hide resolved
@Schmiddiii
Copy link
Collaborator Author

To be honest, I think the suggestive highlighting of the button is too much.

@TheEvilSkeleton
Copy link
Contributor

That's normal design for GNOME apps that have an empty startup nowadays; Loupe, Bottles, etc. do that

data/resources/ui/feed_page.ui Outdated Show resolved Hide resolved
@Schmiddiii
Copy link
Collaborator Author

Should we maybe do a similar thing in the filters and subscriptions pages?

@TheEvilSkeleton
Copy link
Contributor

Absolutely! I didn't think of that, but yes, that would be great

data/resources/ui/filter_page.ui Outdated Show resolved Hide resolved
@TheEvilSkeleton
Copy link
Contributor

To be honest, I think the suggestive highlighting of the button is too much.

About that, if you don't like how it looks, then you can always remove the style. You maintain and develop it after all

data/resources/ui/filter_page.ui Outdated Show resolved Hide resolved
data/resources/ui/subscription_page.ui Outdated Show resolved Hide resolved
data/resources/ui/feed_page.ui Outdated Show resolved Hide resolved
Copy link
Contributor

@TheEvilSkeleton TheEvilSkeleton left a comment

Choose a reason for hiding this comment

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

LGTM

@Schmiddiii Schmiddiii merged commit d614993 into master Mar 27, 2023
@Schmiddiii Schmiddiii deleted the feed-subscribe-button branch March 27, 2023 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants