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

Add AboutDialogHandler #609

Merged
merged 7 commits into from
Feb 10, 2021
Merged

Add AboutDialogHandler #609

merged 7 commits into from
Feb 10, 2021

Conversation

atharva-18
Copy link
Contributor

Reference to #371

Summary:

  1. Added about dialog to GazeboDrawer.qml
  2. Added handler for the above dialog.
  3. Changed ListElement QML type's action attribute to actionElement.

Screenshots:
Screenshot from 2021-02-03 22-30-54
Screenshot from 2021-02-03 22-31-20
Screenshot from 2021-02-03 22-32-04

Signed-off-by: atharva-18 <[email protected]>
@atharva-18 atharva-18 requested a review from chapulina as a code owner February 3, 2021 17:33
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Feb 3, 2021
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

src/gui/AboutDialogHandler.cc Outdated Show resolved Hide resolved
src/gui/AboutDialogHandler.hh Outdated Show resolved Hide resolved
Signed-off-by: atharva-18 <[email protected]>
Signed-off-by: atharva-18 <[email protected]>
@chapulina chapulina added close the gap Features from Gazebo-classic GUI Gazebo's graphical interface (not pure Ignition GUI) labels Feb 3, 2021
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

src/gui/CMakeLists.txt Show resolved Hide resolved
src/gui/resources/GazeboDrawer.qml Outdated Show resolved Hide resolved
src/gui/resources/GazeboDrawer.qml Show resolved Hide resolved
src/gui/AboutDialogHandler.cc Outdated Show resolved Hide resolved
"</td>"
"<td>"
"<a href='https://ignitionrobotics.org/libs/gazebo' "
"style='text-decoration: none; color: #f58113'>"
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know if there's a way, but it would be cool if these links used the Material.accent color. Don't worry if that's not straightforward, I think it's also ok to leave it as is.

Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution, the dialog looks great and this also fixes the menu, which was broken! 👍

src/gui/resources/GazeboDrawer.qml Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel close the gap Features from Gazebo-classic GUI Gazebo's graphical interface (not pure Ignition GUI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants