Skip to content
This repository was archived by the owner on Sep 21, 2022. It is now read-only.

windowSize is needed because by default it is too small#184

Closed
varya wants to merge 1 commit intogemini-testing:masterfrom
varya:master
Closed

windowSize is needed because by default it is too small#184
varya wants to merge 1 commit intogemini-testing:masterfrom
varya:master

Conversation

@varya
Copy link
Copy Markdown

@varya varya commented Jul 20, 2015

windowSize is needed to be in a default config because without it the screenshots are taken in too small size, which might be confusing.
See: https://ru.bem.info/forum/502/

@varya
Copy link
Copy Markdown
Author

varya commented Jul 27, 2015

@SevInf ping :-)

@SevInf
Copy link
Copy Markdown
Contributor

SevInf commented Jul 28, 2015

@varya I don't think windowSize is required for that particular example in README.

@SevInf
Copy link
Copy Markdown
Contributor

SevInf commented Jul 28, 2015

Maybe, we need a separate document with common problems & solutions.

@varya
Copy link
Copy Markdown
Author

varya commented Jul 28, 2015

It is indeed a good idea to have "common problems" page. But if the problem is so common that all the users face it, this must be a signal to fix the problem.

@levonet levonet added the review label Sep 8, 2015
@j0tunn
Copy link
Copy Markdown
Contributor

j0tunn commented Nov 6, 2015

@varya rebase please. I think we can just merge it

@SwinX
Copy link
Copy Markdown
Contributor

SwinX commented Nov 23, 2015

@varya ping?

1 similar comment
@SwinX
Copy link
Copy Markdown
Contributor

SwinX commented Dec 1, 2015

@varya ping?

@varya
Copy link
Copy Markdown
Author

varya commented Dec 1, 2015

Rebased

@SwinX
Copy link
Copy Markdown
Contributor

SwinX commented Feb 29, 2016

Setting windowSize option for each browser alreadt described in browser settings section of config overview. No need to put it explicitly in global scope in per-browser config example.

@SwinX SwinX closed this Feb 29, 2016
@levonet levonet removed the review label Feb 29, 2016
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.

5 participants