Skip to content

Conversation

klutvott123
Copy link
Member

from "Initialising" to "Waiting for connection" to let people know that a connection to the FC is needed for it to work.
Changed telemetry version message to "Use TOOLS menu" to fit the smaller screens.

from "Initialising" to "Waiting for connection" to let people know that a connection to the FC is needed for it to work.
Changed telemetry version message to "Use TOOLS menu" to fit the smaller screens.
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@0crap 0crap left a comment

Choose a reason for hiding this comment

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

Will "Waiting for connection" fit the smaller screens?

@klutvott123
Copy link
Member Author

Fits nicely on the QX7 👍
screenshot_x7_20-05-11_20-14-28

@0crap
Copy link
Contributor

0crap commented May 12, 2020

Looks great! These two small text changes make it much better for the smaller screens.

@mikeller mikeller added this to the 1.5 milestone May 12, 2020
@mikeller mikeller removed this from the 1.5 milestone May 12, 2020
@0crap
Copy link
Contributor

0crap commented May 12, 2020

It's disappointing not to see this in milestone 1.5
@mikeller can you explain why if I may ask?

@mikeller
Copy link
Member

@0crap: Trying to avoid getting even more delay due to feature creep - the reality is that there is always a 'very nice next feature', and waiting for it will delay any release infinitely.

@mikeller mikeller merged commit 66d4900 into betaflight:master May 13, 2020
@0crap
Copy link
Contributor

0crap commented May 13, 2020

@mikeller I fully understand that general remark. But in this case it's not a new feature or something like that. Just make the huge overhaul of 1.5.0 better understandable for the public when they first see it.
Especially because there is no RC phase I would have added this before.
But again, I understand you have more on your mind besides this. Just IMHO.

@klutvott123 klutvott123 deleted the change-init-message branch October 24, 2021 19:34
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.

3 participants