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

Max Size is too inflexible on ultrawide monitors #579

Closed
Tomcc opened this issue Sep 7, 2020 · 2 comments
Closed

Max Size is too inflexible on ultrawide monitors #579

Tomcc opened this issue Sep 7, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@Tomcc
Copy link

Tomcc commented Sep 7, 2020

Describe the bug

First of all I'm really not sure if this is a bug or a feature request... but if you squint you could see it as an accessibility problem for widescreen users, so it may be a bug :)

When using an ultrawide monitor (eg. 5122x1440), there is no ideal setting for the window size.
The default 80% is e x t r e m e l y w i d e, and reducing it to a % where it would make sense causes the height to be too small and needlessly hide rows.

Screenshots / video

This is what 80% looks like:
Screen Shot 2020-09-07 at 00 53 28
Most of the space is wasted and it forces me to turn my head to look at the options all the way on the left.

And this is what the width I would like (15%) looks like:
Screen Shot 2020-09-07 at 00 54 01

Now the width is ok, but it's not high enough and it's hiding rows for no reason.

I think the issue here is that width and height should be decoupled... and personally I think that defining the width using a % doesn't work well if someone uses many different screens with their Mac, the panel's size will be all over the place.
For my use case where I want a column of options, setting a max size in pixels would be much nicer.

Steps to reproduce the bug

  1. Find an ultrawide monitor
  2. Try to use Alt Tab and fiddle with the Max Size %
  3. See the above
@Tomcc Tomcc added the bug Something isn't working label Sep 7, 2020
@lwouis lwouis added enhancement New feature or request and removed bug Something isn't working labels Sep 7, 2020
@lwouis
Copy link
Owner

lwouis commented Sep 7, 2020

Hey @Tomcc! Thanks for sharing this feedback!

AltTab currently doesn't support your use-case. Thus I'm flagging this as an enhancement.

I think you're right that the best way to support this layout is to split Max size on screen into Max width on screen and Max height on screen.

I'll also take the opportunity to:

  • Rename Min/Max windows per row into Window min/max width in row for more clarity on what these sliders d
  • Rename Rows of windows into Rows of thumbnails, as this setting only applies if to thumbnails
  • Make it so checking Hide window thumbnails disables Rows of thumbnails to illustrate the relationship of these 2 sliders
  • Make it so it's impossible to have a Window min width in row bigger than Window max width in row

Actually, I've done the work already. Here is a video showcase. Let me know what you think.

It will ship in the next release

@Tomcc
Copy link
Author

Tomcc commented Sep 8, 2020

Wow, I don't think anyone ever fixed a bug I reported with a video showcase, I don't even know what to say :)

Thanks for fixing this so quickly! It looks like it does allow me to get the look I wanted, I'll try it when the new version is out.

And thanks again for making Alt Tab, I was using Contexts but I had a few issues with it. This one is so much better!

@lwouis lwouis closed this as completed in 6e2e5b4 Sep 8, 2020
lwouis pushed a commit that referenced this issue Sep 8, 2020
# [6.3.0](v6.2.0...v6.3.0) (2020-09-08)

### Bug Fixes

* adobe audition windows were not showing up (closes [#581](#581)) ([6edced0](6edced0))
* crash from appcenter in rare scenario ([c49a2bc](c49a2bc)), closes [#1](#1)
* load app badges asynchronously to avoid system lag (closes [#563](#563)) ([29eff03](29eff03))
* prevent rare crash seen in app center ([6ca58b1](6ca58b1))

### Features

* show minimized/hidden windows last in the list (closes [#289](#289)) ([4fea943](4fea943))
* split max screen size preference into width/height (closes [#579](#579)) ([6e2e5b4](6e2e5b4))
* update es, ja, ko, nl, pt, pt-br localizations ([af5ed9b](af5ed9b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants