Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

My Homepage should be enabled only when Brave Starts with/A new tab shows has homepage set in it #5451

Closed
srirambv opened this issue Nov 7, 2016 · 2 comments
Labels

Comments

@srirambv
Copy link
Collaborator

srirambv commented Nov 7, 2016

My Homepage should be enabled only when Brave Starts with/A new tab shows has homepage set in it

  • Brave Version: master
@bsclifton
Copy link
Member

bsclifton commented Nov 7, 2016

I'd have to disagree with this one after working through a previous (similar) example. cc: @bradleyrichter and @bbondy. I'm going to remove the milestone and close, but let's talk about it (and reopen if feeling strongly)

My reasoning:

  1. You can show the home button and use it without setting "Brave Starts with/A new tab shows".
  2. I had done something similar which I thought made sense UX-wise and ended up reverting it after discussion (see Split apart "always show bookmarks toolbar" and the bookmarks toolbar display mode #4944 for the revert). The change I had done disabled choosing the bookmarks toolbar types you want unless you chose to enable "always show bookmarks toolbar".

@bradleyrichter
Copy link
Contributor

Here is another reason to keep it separated:

image

If the user is not able to define their home page address, then we would need to disable this keyboard short cut. I suspect the keyboard-nav fans would disapprove.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants