Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Hide explore section if logged in#1024

Open
flxholle wants to merge 1 commit intoslapperwan:masterfrom
quwepiro:master
Open

Hide explore section if logged in#1024
flxholle wants to merge 1 commit intoslapperwan:masterfrom
quwepiro:master

Conversation

Copy link
Contributor

flxholle commented Aug 30, 2020 *
edited
Loading

Personally, I do not use the explore section and it makes the navigation drawer longer, than it would be needed.
I think this is only useful if the user is not logged in.

Copy link
Collaborator

maniac103 commented Sep 18, 2020

I don't see the relationship between the explore section and the 'user is logged in status'. Personally, I use both trending list and (occasionally) the blog list despite being logged in. The only thing that probably isn't too useful for logged in users is the global GH timeline ... but at the same time, I wonder why there should be a problem in keeping it. The only item that comes after these items is 'Settings', and that surely isn't used that often?

Copy link
Contributor Author

flxholle commented Sep 19, 2020

Hi @maniac103
I see your point. I think this is personal decision, I do not use any of the three features and I often click on them by accident, so I removed them for myself. If you do not want this change you can close this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants