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

minimum viewport size for desktop-gui needs to be increased #869

Closed
jennifer-shehane opened this issue Nov 3, 2017 · 6 comments · Fixed by #4737
Closed

minimum viewport size for desktop-gui needs to be increased #869

jennifer-shehane opened this issue Nov 3, 2017 · 6 comments · Fixed by #4737
Labels
good first issue Good for newcomers pkg/desktop-gui This is due to an issue in the packages/desktop-gui directory type: unexpected behavior User expected result, but got another

Comments

@jennifer-shehane
Copy link
Member

jennifer-shehane commented Nov 3, 2017

The nav overlaps with the project name displaying in lefthand corner.

screen shot 2017-11-03 at 2 11 42 pm

I think this can be fixed in the packages/desktop-gui css, but we could also set a minWidth on the window itself here https://github.com/cypress-io/cypress/blob/code-of-conduct/packages/server/lib/gui/windows.coffee#L129

@jennifer-shehane jennifer-shehane added good first issue Good for newcomers stage: ready for work The issue is reproducible and in scope type: unexpected behavior User expected result, but got another labels Nov 3, 2017
@brian-mann
Copy link
Member

brian-mann commented Nov 3, 2017 via email

@jennifer-shehane jennifer-shehane added the pkg/server This is due to an issue in the packages/server directory label Nov 3, 2017
@chrisbreiding
Copy link
Contributor

Is increasing the minimum width the right solution? What if someone has a project name longer than cypress-example-kitchensink?

I think a better solution is to make the desktop-gui more responsive to width changes. Right now, there are too many hard-coded values for how tall certain elements are assumed to be. We could utilize flexbox to help with situations like this.

@jennifer-shehane
Copy link
Member Author

^ that's why I mentioned css in desktop-gui at first. I'd have to look back at what we're doing for the nav to remember, but most definitely flexbox is a good solution here.

@chrisbreiding chrisbreiding self-assigned this Nov 15, 2017
@chrisbreiding chrisbreiding added pkg/desktop-gui This is due to an issue in the packages/desktop-gui directory and removed good first issue Good for newcomers pkg/server This is due to an issue in the packages/server directory labels Nov 15, 2017
@jennifer-shehane jennifer-shehane added the good first issue Good for newcomers label Jul 13, 2019
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jul 17, 2019

The code for this is done in cypress-io/cypress#4737, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot cypress-bot bot removed stage: needs review The PR code is done & tested, needs review stage: work in progress labels Jul 17, 2019
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jul 29, 2019

Released in 3.4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers pkg/desktop-gui This is due to an issue in the packages/desktop-gui directory type: unexpected behavior User expected result, but got another
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants