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

Screen size usability #354

Closed
Neved4 opened this issue Sep 15, 2024 · 2 comments
Closed

Screen size usability #354

Neved4 opened this issue Sep 15, 2024 · 2 comments

Comments

@Neved4
Copy link

Neved4 commented Sep 15, 2024

Describe the problem

More a discussion than an issue, I’m interested in your thoughts on this one.

After 266645a Cinny's default size is 1280x905. This seems large compared to the previous one.

I can see that the intent is to make better use of screen real state, and I’m wondering if a different resolution might be better:

  1. 1280x905 feels unusual, is there a specific reasoning behind it?
  2. A resolution like 1280x720 might align better with common resolutions.
    Cinny initially having smaller size made it feel friendly and minimal, though this might be subjective.
  3. After Update screen size to more usable #347 Cinny will remember screen size and position,
    so everyone will be able to adjust app size to their personal preference.
  4. Default positions like x: 320, y: 160 (Discord's default) could improve initial screen placement,
    as center seems to currently bleed through the Dock in macOS.

Describe the solution you'd like

Maybe we could add these after #347:

"windows": [
  {
    "title": "Cinny",
    "width": 1280,
    "height": 720,
    "x": 320,
    "y": 160,
    // ...
  }
]
@kfiven
Copy link
Collaborator

kfiven commented Sep 22, 2024

The reason I increased it because on the login screen we had to do the scroll or resize the window to see the all of the options.

I don't know if it's unusual but all I did is increased the old values by 25%.

Although I think this won't be an issue once we merge #327.

@Neved4
Copy link
Author

Neved4 commented Sep 22, 2024

Thanks much for the explanation I understand it now

@Neved4 Neved4 closed this as completed Sep 22, 2024
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

2 participants