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

style: update state panel plugin #8846

Merged

Conversation

KhalilSelyan
Copy link
Contributor

@KhalilSelyan KhalilSelyan commented Sep 11, 2024

Description

Some minimal updates to the style to correct the look overall according to the qt5ct PR as well

Related links

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@KhalilSelyan KhalilSelyan self-assigned this Sep 11, 2024
@github-actions github-actions bot added the component:common Common packages from the autoware-common repository. (auto-assigned) label Sep 11, 2024
@KhalilSelyan KhalilSelyan requested a review from xmfcx September 11, 2024 02:09
Copy link

github-actions bot commented Sep 11, 2024

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@xmfcx xmfcx added the tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Sep 11, 2024
Copy link

codecov bot commented Sep 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 115 lines in your changes missing coverage. Please review.

Project coverage is 27.99%. Comparing base (0d55025) to head (3bd34d7).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...er4_state_rviz_plugin/src/autoware_state_panel.cpp 0.00% 36 Missing ⚠️
...mmon/tier4_state_rviz_plugin/src/custom_slider.cpp 0.00% 24 Missing ⚠️
...mmon/tier4_state_rviz_plugin/src/custom_button.cpp 0.00% 22 Missing ⚠️
...e_rviz_plugin/src/custom_segmented_button_item.cpp 0.00% 19 Missing ⚠️
...er4_state_rviz_plugin/src/custom_toggle_switch.cpp 0.00% 14 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8846      +/-   ##
==========================================
- Coverage   28.01%   27.99%   -0.03%     
==========================================
  Files        1319     1319              
  Lines       98635    98702      +67     
  Branches    39789    39786       -3     
==========================================
- Hits        27631    27628       -3     
- Misses      70949    70982      +33     
- Partials       55       92      +37     
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 28.02% <ø> (+0.01%) ⬆️ Carriedforward from 003808b

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@xmfcx
Copy link
Contributor

xmfcx commented Sep 11, 2024

I will review this along with the following PR:

@xmfcx xmfcx force-pushed the style/update-state-panel-plugin branch from 430f75c to ca422f5 Compare September 11, 2024 15:08
@xmfcx
Copy link
Contributor

xmfcx commented Sep 11, 2024

Could you add pressed state to these buttons as well?
image

@xmfcx
Copy link
Contributor

xmfcx commented Sep 11, 2024

image

these should also have pressed state

@xmfcx
Copy link
Contributor

xmfcx commented Sep 11, 2024

Please apply the hovered colors to the switch
image

@xmfcx xmfcx force-pushed the style/update-state-panel-plugin branch from d432432 to b59e9a9 Compare September 12, 2024 09:31
@xmfcx xmfcx force-pushed the style/update-state-panel-plugin branch 2 times, most recently from 869c03e to 95dff2f Compare September 12, 2024 15:13
@xmfcx xmfcx marked this pull request as draft September 12, 2024 16:20
@xmfcx xmfcx marked this pull request as ready for review September 17, 2024 11:40
M. Fatih Cırıt and others added 11 commits September 17, 2024 14:40
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
Signed-off-by: M. Fatih Cırıt <[email protected]>
@xmfcx xmfcx force-pushed the style/update-state-panel-plugin branch from 841d2e0 to 3bd34d7 Compare September 17, 2024 11:40
Copy link
Contributor

@xmfcx xmfcx left a comment

Choose a reason for hiding this comment

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

image

Current state.

With everything else:
image

@xmfcx xmfcx merged commit 28a389b into autowarefoundation:main Sep 17, 2024
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:common Common packages from the autoware-common repository. (auto-assigned) tag:run-build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants