Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Restrict initial window size on big screens - Closes #221 #226

Merged
merged 3 commits into from
May 17, 2017

Conversation

slaweet
Copy link
Contributor

@slaweet slaweet commented May 15, 2017

To 0.5 of width and 0.7 of height.

Closes #221

@slaweet slaweet added the bug label May 15, 2017
@slaweet slaweet self-assigned this May 15, 2017
@slaweet slaweet requested a review from karmacoma May 15, 2017 16:22
to 0.5 of width and 0.7 of height
@slaweet slaweet force-pushed the 221-default-window-size branch from b3b0fe1 to b05af6e Compare May 15, 2017 16:24
@slaweet slaweet changed the title Restrict initial window size on big screens Restrict initial window size on big screens - Closes #221 May 16, 2017
app/main.js Outdated
width: width - 250,
height: height - 150,
width: width > 2000 ? Math.floor(width * 0.5) : width - 250),
height: height > 1000 ? Math.floor(height * 0.7) : height - 150),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check parentheses.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Stupid mistake. But made me realise that this file wasn't under eslint.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants