Skip to content

Make trending gifs optional with a menu option#167

Open
popthestack wants to merge 2 commits intomasterfrom
feature/making-trending-gifs-optional
Open

Make trending gifs optional with a menu option#167
popthestack wants to merge 2 commits intomasterfrom
feature/making-trending-gifs-optional

Conversation

@popthestack
Copy link
Collaborator

Make trending gif fetching optional.

@popthestack
Copy link
Collaborator Author

One thing: this doesn't remove the little button that refreshes the trending list, so you can still load them. The menu item might be confusing because it just hides them on start?

@adamweeks
Copy link
Owner

@popthestack Could we make the config instead be to choose the default home screen? I'm assuming you want favorites to be the home screen instead.

@popthestack
Copy link
Collaborator Author

@adamweeks I like that idea, I'll work on it.

@popthestack popthestack changed the title Add trending gifs optional with a menu option Make trending gifs optional with a menu option Feb 23, 2019
adamweeks
adamweeks previously approved these changes Feb 25, 2019
@popthestack
Copy link
Collaborator Author

🎉 I will make more improvements to this later on.

@adamweeks
Copy link
Owner

@popthestack looks like tests are failing with this one 😢

@adamweeks adamweeks dismissed their stale review February 25, 2019 15:32

Due to failed tests

@popthestack
Copy link
Collaborator Author

@adamweeks noooo

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.

2 participants