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

GPII-2404: Changed windows' display settings handler to retrieve all screen resolutions #121

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

klown
Copy link
Member

@klown klown commented May 8, 2017

@kaspermarkus, @javihernandez

The modifiications to the window settings handler analogous to the linux pull request.

…creen resolutions

- Modified the display settings handler to get both the current and all available
screen resolutions in one JSON object.
- Refactored the handler into a fluid component.
- Updated the unit tests in light of the above.
@gpii-bot
Copy link

gpii-bot commented May 8, 2017

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

…creen resolutions

Reverted getScreenResolution() to return an object with just width and height, but
modified getImpl() to add a "screen-resolution" property.
@gpii-bot
Copy link

CI job passed.

…resolutions

Better way to set current screen resolution return value.  Also, cleaned up some
comments.
@gpii-bot
Copy link

CI job passed.

@gpii-bot
Copy link

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@klown
Copy link
Member Author

klown commented Jun 26, 2017

@amatas, according to the console log, the VM creation timed out. No tests were run. Any idea why?

@amatas
Copy link

amatas commented Jun 27, 2017

ok to test

@gpii-bot
Copy link

CI job passed.

@amatas
Copy link

amatas commented Jun 27, 2017

@klown sorry but seems that the creation of the VM took so much time to install some things that the Jenkins aborted the task. Perhaps it was because the host was very busy with other tasks. Now the tests passed.

@klown
Copy link
Member Author

klown commented Jun 27, 2017

@amatas no problem -- it's not your fault. Thanks for re-running it!

@gpii-bot
Copy link

gpii-bot commented Jul 5, 2017

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@klown
Copy link
Member Author

klown commented Jul 12, 2017

@amatas Tapping your shoulder...

@amatas
Copy link

amatas commented Jul 12, 2017

ok to test

@gpii-bot
Copy link

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@klown
Copy link
Member Author

klown commented Jul 12, 2017

ok to test

@gpii-bot
Copy link

CI job passed.

@gpii-bot
Copy link

gpii-bot commented Aug 4, 2017

CI job passed.

@gpii-bot
Copy link

CI job passed.

@gpii-bot
Copy link

CI job passed.

Modifed the "getAllScreenResolutions" to drop the "dmPels" prefix.
@gpii-bot
Copy link

CI job passed.

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/389/

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/393/

@klown
Copy link
Member Author

klown commented Feb 20, 2018

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/395/

Merge upstream master GPII windows branch into GPII-2404.
@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/415/

@klown
Copy link
Member Author

klown commented Mar 12, 2018

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/417/

@gpii-bot
Copy link

gpii-bot commented Apr 2, 2018

CI job passed: https://ci.gpii.net/job/windows-tests/444/

@klown
Copy link
Member Author

klown commented Apr 5, 2018

ok to test

@gpii-bot
Copy link

gpii-bot commented Apr 5, 2018

CI job passed: https://ci.gpii.net/job/windows-tests/452/

@gpii-bot
Copy link

CI job failed: https://ci.gpii.net/job/windows-tests/469/

@klown
Copy link
Member Author

klown commented Apr 25, 2018

ok to test

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/470/

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/493/

@klown
Copy link
Member Author

klown commented Jun 4, 2018

ok to test

@gpii-bot
Copy link

gpii-bot commented Jun 4, 2018

CI job failed: https://ci.gpii.net/job/windows-tests/512/

@klown
Copy link
Member Author

klown commented Jun 5, 2018

ok to test

@gpii-bot
Copy link

gpii-bot commented Jun 5, 2018

CI job failed: https://ci.gpii.net/job/windows-tests/519/

@klown
Copy link
Member Author

klown commented Jun 6, 2018

ok to test

@gpii-bot
Copy link

gpii-bot commented Jun 6, 2018

CI job failed: https://ci.gpii.net/job/windows-tests/522/

@gpii-bot
Copy link

gpii-bot commented Jun 7, 2018

CI job passed: https://ci.gpii.net/job/windows-tests/527/

@gpii-bot
Copy link

gpii-bot commented Jun 8, 2018

CI job passed: https://ci.gpii.net/job/windows-tests/532/

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

Successfully merging this pull request may close these issues.

4 participants