All Integration > QuickOpen Unit tests fail

[Mark-Simulacrum]

All 3 tests fail with:

• can open a file and jump to a line, centering that line on the screen
  • Error: Expected 374 to be less than 248.39999999999998.
• can open a file and jump to a line and column, centering that line on the screen
  • Error: Expected 374 to be less than 248.39999999999998.
• can directly open a file in a given line and column, centering that line on the screen
  • Error: Expected 287 to be less than 248.39999999999998.

Perhaps these are rounding issues?

Tested on Ubuntu 64bit 14.04.

[ingorichter]

The values are too far off for a rounding issue. What is your screen size? I remember that we had something similar a while ago where the window size was way bigger so that lines didn't wrap anymore. Perhaps this is related.

[Mark-Simulacrum]

@ingorichter Hmm, not really sure how to find out screen size. I have 1920 x 1080 resolution though, not sure if that helps.

Also not sure how screen size would relate, since as far as I can tell, the integration tests open a new Brackets window; which would seem to prevent any screen size related issues. Perhaps I am wrong there, though.

[mackenza]

These are the results I see on Ubuntu 14.04 64bit as well. My screen is split across two monitors (an LCD and the built-in LCD of the laptop).

[mackenza]

I am not sure if this is related to the behaviour on Linux that when a new Brackets instance is opened, it doesn't remember screen position or size.

[redmunds]

the behaviour on Linux that when a new Brackets instance is opened,
it doesn't remember screen position or size.

That's a known issue. Know any native Linux devs?

[mackenza]

Ryan already asked me... sadly, no. I just thought maybe it might be
related to the screen centering in the unit tests not recording the right
values.

On Tue, Sep 2, 2014 at 8:59 PM, Randy Edmunds notifications@github.com
wrote:

the behaviour on Linux that when a new Brackets instance is opened,
it doesn't remember screen position or size.

That's a known issue. Know any native Linux devs?

—
Reply to this email directly or view it on GitHub
#8889 (comment).