Skip to content

Conversation

@haslinghuis
Copy link
Member

  • Splits header so elements can fit on one row.
  • Fixes i18n message.

Screenshot from 2021-10-10 20-36-01

We should have enough room to fit all elements on one row if we use an extra header in the first column and adjust width (about 8% + 12%).
Also think we need to adjust the second column to go down on smaller screens at around 600px to fit other labels too.

@haslinghuis haslinghuis added this to the 10.8.0 milestone Oct 13, 2021
@haslinghuis haslinghuis self-assigned this Oct 13, 2021
ctzsnooze
ctzsnooze previously approved these changes Oct 14, 2021
asizon
asizon previously approved these changes Oct 14, 2021

.tab-pid_tuning .subtab-pid .cf_column {
min-width: 472px;
min-width: 600px; /* 472px; */
Copy link
Member

Choose a reason for hiding this comment

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

remove comment?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed!

Copy link
Member Author

Choose a reason for hiding this comment

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

Squashed commits.

@haslinghuis haslinghuis dismissed stale reviews from asizon and ctzsnooze via 3b0ac87 October 14, 2021 16:55
Fix sonar bug

Remove comment
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@blckmn
Copy link
Member

blckmn commented Nov 9, 2021

AUTOMERGE: (PASS)

  • github identifies PR as mergeable -> PASS
  • assigned to a milestone -> PASS
  • cooling off period lapsed -> PASS
  • commit count less or equal to three -> PASS
  • Don't merge label NOT found -> PASS
  • at least one RN: label found -> PASS
  • Tested label found -> PASS
  • assigned to an approver -> PASS
  • approver count at least three -> PASS

@blckmn blckmn merged commit dd183f0 into betaflight:master Nov 9, 2021
@haslinghuis haslinghuis deleted the split_header branch November 9, 2021 12:47
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.

6 participants