RSSI to display 100 instead of SYM_MAX #10560
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
CURRENT BEHAVIOUR
The current way the RSSI is displayed is such that the RSSI takes up two characters. Whilst this is OK for most OSD systems which have a dedicated icon for RSSI and LQ, it causes alignement issues with regards to the values when used with the O3 system for example. As the LQ hovers around 100 most of the time and the RSSI is usually lower than 100, there is logic behind the value taking up 2 spaces and being replaced by the MAX symbol when it is 100.
But this is not ideal when you would like to align those values to the symbol as well as the number
DESIRED BEHAVIOUS
I think that the MAX symbol should not be used. Istead, the value should be shown as 100 when it is 100, same as the LQ. This way both the LQ and the RSSI, which are most frequently used together will both be showing with the same total number of characters, i.e. 4 in total, 1 for the signal bars icon and 3 for the value. This way users can correclty align the two signal icons one under the other and the values can be aligned to the right.
EXAMPLE
In other words, like this:
Specifically, for users of the O3 system, which lacks separate icons for RSSI and LQ can use the custom fields to display what they would like as plain text next to the respective line.
OTHER COMMENTS
Other systems would benefit from this as well, as the value will be correclty alligned when placed one under the other.
In my case the LQ is the top and the the RSSI is at the bottom and I added those lables as text next to the value
ADDITIONAL WORK
The only thing that would need to be done would be to adjust the lenght in the configurator as well.
Feel free to let me know if you think I've made a mistake, but this just my way of thinking about this.