Show Open New Window button in small window
Pull Request Type
- [ ] Bugfix
- [X] Feature Implementation
- [ ] Documentation
- [ ] Other
Related issue
Closes #4155
Description
Removes the CSS code that hides the Open New Window button when the window is small.
Screenshots
Before
After
Testing
- Reduce the window size
- See the the Clone Window button appears
Desktop
- OS: Linux
- OS Version: Ubuntu 24
- FreeTube version: v0.23.12 Beta
Additional context
I updated the button order in TopNav.vue to place the Clone button next to the navigation arrows, similar to the large window layout.
Before, the Clone button would be positioned to the left of the Search button on smaller screens. This change does not affect large window layouts.
Let me know your thoughts on this!
Not sure if we should still hide it on very small screens though. @PikachuEXE @absidue thoughts?
It's from https://github.com/FreeTubeApp/FreeTube/commit/81a571e7c9f6f407e78bdb34eea77c6e2a80ccea I dunno why it's committed, and I don't use small width windows so I have no objection (maybe you should ask mobile users
Nobody on a mobile device ever complained about it, because it doesn't make sense to have multiple windows on a mobile device, the only people that complain about it are people using tiling window managers with so many windows on screen at the same time that the window is so small that FreeTube has to fallback to the mobile UI (IIRC @efb4f5ff-1298-471a-8973-3d47447115dc opened the issue on behalf of those complaining tiling window manager users).
If this is merged it will require a change in FreeTubeAndroid to add an IS_ANDROID build guard to remove the button, as it doesn't and shouldn't work in FreeTubeAndroid (previously that change wasn't necessary as it was hidden by code that is being removed in this PR).
I dunno why it's committed
I looked through the issues starting when the button was introduced until it was changed to be hidden and it doesn't address any issue. I think that Preston thought that its logical to not include it on smaller screens
(IIRC @efb4f5ff-1298-471a-8973-3d47447115dc opened the issue on behalf of those complaining tiling window manager users).
Yup thats correct
@absidue do you think that we shouldn't address this issue?
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.