Skip to content

Feature/finished/iia 1350 close prop panel#243

Merged
stenkarl merged 69 commits intoikmdev:mainfrom
dholubek:feature/finished/IIA-1350-close-prop-panel
Feb 3, 2025
Merged

Feature/finished/iia 1350 close prop panel#243
stenkarl merged 69 commits intoikmdev:mainfrom
dholubek:feature/finished/IIA-1350-close-prop-panel

Conversation

@dholubek
Copy link
Contributor

@dholubek dholubek commented Feb 3, 2025

https://ikmdev.atlassian.net/browse/IIA-1350

window before clicking submit:
image

window after clicking submit:
image

window after clicking close properties panel:
image

Doug Holubek and others added 30 commits August 29, 2024 08:58
@dholubek dholubek marked this pull request as draft February 3, 2025 16:57
@dholubek dholubek self-assigned this Feb 3, 2025
@dholubek dholubek marked this pull request as ready for review February 3, 2025 17:01
…om:dholubek/komet into feature/finished/IIA-1350-close-prop-panel
@dholubek dholubek requested a review from stenkarl February 3, 2025 17:35
Copy link
Contributor

@dukke dukke left a comment

Choose a reason for hiding this comment

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

The text might be off... It might be off already in the Figma... We are editing not adding but the text says: "Semantic Details Added"

Copy link
Contributor

@bhharsh13 bhharsh13 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

<RowConstraints minHeight="10.0" prefHeight="30.0" vgrow="SOMETIMES" />
</rowConstraints>
<children>
<Label alignment="TOP_CENTER" styleClass="lidr-device-title-label" text="Semantic Details Added" textAlignment="CENTER" GridPane.columnIndex="1">
Copy link
Contributor

Choose a reason for hiding this comment

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

I left this comment in the PR but leaving here for better context:
The text might be off... It might be off already in the Figma... We are editing not adding but the text says: "Semantic Details Added"

Copy link
Contributor

Choose a reason for hiding this comment

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

We discussed as a team and we're going to leave it for now since it matches the figma.

@stenkarl stenkarl merged commit d0da91c into ikmdev:main Feb 3, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants