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

Make the tabs tray full screen (behave as similarly to old Fennec as possible) #73

Closed
abhijitvalluri opened this issue Sep 6, 2020 · 3 comments

Comments

@abhijitvalluri
Copy link
Collaborator

What is the user problem or growth opportunity you want to see solved?

Make the tabs tray full screen and only have full screen or hidden modes. No sliding and opening half way etc. Make it similar to how old Fennec behaved. I do not see a point for the tabs tray to be collapsed or anything. It does not gain us anything and is primarily only for the coolness and showy factor. It is not like you can do something with the partially visible tab in the background.

How do you know that this problem exists today? Why is this important?

Current implementation of the tab tray is like a pop up view. It is quite annoying and it wastes user's time as you have to open the tab tray fully after clicking on the tabs button. So an extra unnecesary step is needed.

Who will benefit from it?

Everyone who likes a quick interface and prefers function over form.

A demo of the potential final result

I made a quick and dirty hack to the code to see how it might look. This is what I got! See how the tab tray is completely full screen? It covers up even the URL bar. In the official version it is impossible to cover the URL bar, so the tabs tray wastes some space needlessly.

This version allows only full screen or closed, no half open or any such thing.

Any feedback? Please comment below!

@abhijitvalluri
Copy link
Collaborator Author

We can easily tweak this to avoid wasting space for the top icons as well. Worth a shot.

@abhijitvalluri
Copy link
Collaborator Author

This is resolved by #69. Check this for reference: #69 (comment)

@abhijitvalluri
Copy link
Collaborator Author

Resolved by the merged PR!

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

No branches or pull requests

1 participant