Skip to content

Instead of 400 pixels, make the height relative to the display height #433

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

Closed
a2sheppy opened this issue Aug 5, 2020 · 6 comments
Closed
Labels
enhancement New feature or enhancement to existing functionality

Comments

@a2sheppy
Copy link

a2sheppy commented Aug 5, 2020

Currently, the popup for this menu extra on Mac comes up with a fixed height of 400 pixels. This is so very small and doesn't allow much of a window into the state of my GitHub activities. Instead, it should be based on the display height, taking the workArea.height value and subtracting a small amount to allow it to not just fill the height between menu bar and dock. I fiddled with this a bit and couldn't quite make it work, but didn't spend much time on it either (I tried some shortcuts that just didn't work). Figured I'd file this just in case others are interested in trying to make it happen.

@a2sheppy a2sheppy changed the title Instead of fixed height, make the height relative to the display height Instead of 400 pixels, make the height relative to the display height Aug 5, 2020
@JakeSidSmith
Copy link
Collaborator

It would be nice to be able to resize the height, rather than a fixed value. 🙂 Not sure of the limitation of electron windows though.

@JakeSidSmith
Copy link
Collaborator

And I see you've just created #434 XD 👍

@a2sheppy
Copy link
Author

a2sheppy commented Aug 5, 2020

I've also changed resizable to true in my local fork and everything works fine, though I doubt the size is saved. :)

@JakeSidSmith
Copy link
Collaborator

How is the positioning with resizable? I believe the reason this was not enabled originally was due to issues with the position of the window after resize (or something like that).

@a2sheppy
Copy link
Author

Seems fine to me. You can of course resize the window from the top, so it looks weird, but that doesn't mean it doesn't work just fine.

@fdintino
Copy link

fdintino commented Sep 3, 2020

@JakeSidSmith I'm not so sure…it was resizable before the 3.0 release. It got disabled in b8e8b16, but I'm not sure how intentional that was. Flipping resizable: true doesn't seem to have any ill effects.

@setchy setchy added the enhancement New feature or enhancement to existing functionality label Jan 19, 2024
@setchy setchy closed this as not planned Won't fix, can't repro, duplicate, stale Jun 17, 2024
@setchy setchy removed the wont-fix label Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement to existing functionality
Projects
None yet
Development

No branches or pull requests

5 participants