Skip to content

Mirrord RT toolbar on tracking station#202

Merged
Starstrider42 merged 1 commit into
RemoteTechnologiesGroup:developfrom
Peppie84:tracking_station_ui_flip
Oct 7, 2014
Merged

Mirrord RT toolbar on tracking station#202
Starstrider42 merged 1 commit into
RemoteTechnologiesGroup:developfrom
Peppie84:tracking_station_ui_flip

Conversation

@Peppie84

@Peppie84 Peppie84 commented Oct 7, 2014

Copy link
Copy Markdown
Member

For v0.25.0 compatibility #185

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is an absolute size; can this cause a repeat of #129 for players with different-sized screens or different UI settings?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yea i thought the same but the screen size isn't changing for the ui scaling. i tested it from tiny to huge. Even on x64 ;)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

oh wait :D i'll check if i change the screen resolution.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

hm no looks good, even with screen resolution of 1680x1050

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok then, looks plausible.

I won't be able to playtest for another 3-4 hours, but it would be easier to do that with a merged develop branch. Merging.

Starstrider42 added a commit that referenced this pull request Oct 7, 2014
Mirrored RT toolbar on tracking station
@Starstrider42 Starstrider42 merged commit 03617c9 into RemoteTechnologiesGroup:develop Oct 7, 2014
@Peppie84 Peppie84 deleted the tracking_station_ui_flip branch October 8, 2014 10:40
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.

2 participants