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

Strange behavior when opening popover #11366

Closed
mlqn opened this issue Oct 15, 2023 · 5 comments · Fixed by #11779
Closed

Strange behavior when opening popover #11366

mlqn opened this issue Oct 15, 2023 · 5 comments · Fixed by #11779
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. kind/bug Used when there is a defect / something is not working as it should. status/triage

Comments

@mlqn
Copy link
Contributor

mlqn commented Oct 15, 2023

Description of the bug

The 'Change Page Name' popover doesn't position correctly if the deletion popover is not closed properly.

Steps To Reproduce

popover.mov

Additional Information

No response

@mlqn mlqn added kind/bug Used when there is a defect / something is not working as it should. status/triage area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. labels Oct 15, 2023
@nkylstad nkylstad moved this to 📈 Todo in Team Studio Oct 23, 2023
@ghost ghost self-assigned this Nov 27, 2023
@ghost ghost moved this from 📈 Todo to 👷 In Progress in Team Studio Nov 27, 2023
@ghost ghost moved this from 👷 In Progress to ⚠️ Blocked in Team Studio Nov 27, 2023
@ghost
Copy link

ghost commented Nov 27, 2023

Blocked by the Z-index bug in the DropdownMenu component from the Designsystem. Issue: digdir/designsystemet#1134

@ghost
Copy link

ghost commented Dec 4, 2023

Blocked by #11759

@ghost ghost linked a pull request Dec 4, 2023 that will close this issue
3 tasks
@ghost ghost moved this from ⚠️ Blocked to 🔎 Review in Team Studio Dec 4, 2023
@ghost ghost removed their assignment Dec 4, 2023
@JamalAlabdullah JamalAlabdullah self-assigned this Dec 5, 2023
@ghost ghost closed this as completed in #11779 Dec 13, 2023
@github-project-automation github-project-automation bot moved this from 🔎 Review to 🧪 Test in Team Studio Dec 13, 2023
@nkylstad
Copy link
Member

Tested OK i dev. Was not able to reproduce the issue, and both deleting page and renaming works as expected.
Just a question about the design, was this change moving the delete button out of the context menu discussed with @Annikenkbrathen?

@nkylstad nkylstad moved this from 🧪 Test to 👀 Test feedback in Team Studio Dec 13, 2023
@nkylstad nkylstad assigned ghost Dec 13, 2023
@ghost
Copy link

ghost commented Dec 13, 2023

Tested OK i dev. Was not able to reproduce the issue, and both deleting page and renaming works as expected. Just a question about the design, was this change moving the delete button out of the context menu discussed with @Annikenkbrathen?

Yes, discussed with @Annikenkbrathen 😄

@nkylstad
Copy link
Member

Great! Let's start adding a short comment in the issue about design decisions in our issues as well if something comes up during implementation 😊 Makes it easier for whomever is testing, and it's good to document our decisions 👍 Will suggest it as a general practice going forward in the next retro!

@nkylstad nkylstad moved this from 👀 Test feedback to ✅ Done in Team Studio Dec 13, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. kind/bug Used when there is a defect / something is not working as it should. status/triage
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants