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

Spektrum iX12 adaption of CMS start screen. #5181

Merged

Conversation

AndersHoglund
Copy link
Contributor

Minor cosmetic adaptation to the new Spektrum iX12 Tx. The CMS welcome screen looked awful because of the way iX12 has divided the TM screen into 2 columns. Work around by moving the text up one row.

Looks like this before the change, on DX18 and iX12.
spektrum_dx18_tm_text_cms_before
spektrum_ix12_tm_text_cms_before

After this PR it looks like this
spektrum_dx18_tm_text_cms_after
spektrum_ix12_tm_text_cms_after

@mikeller
Copy link
Member

Shouldn't this be in 3.4, considering it is an improvement, and was submitted after the feature freeze for 3.3 began?

@wind0r
Copy link
Member

wind0r commented Feb 13, 2018

Spektrum CMS got introduced with 3.3. Now with the RC1 People found out that the interface looks strange on some radio. This is a Bugfix for that. Waiting for 3.4 realy sound strange there.

Having a Feature Freeze should still allow us to create smaller improvments and bugfixes which got found in rc phase. I think thats exactly where the rc phase is for ;)

@mikeller
Copy link
Member

All good then.

@mikeller mikeller merged commit 5593b43 into betaflight:master Feb 13, 2018
@AndersHoglund
Copy link
Contributor Author

Thank you. Was out and away for awhile, missed the argue-ing.

@AndersHoglund AndersHoglund deleted the spektrum_ix12_cms_adaption branch February 24, 2018 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants