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

Fix dropdown positioning issue inside auro-dialog #190

Merged
merged 3 commits into from
Jun 10, 2024

Conversation

fajar-apri-alaska
Copy link
Contributor

Alaska Airlines Pull Request

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Resolves: #187

Summary:

  • This PR addresses the positioning issue of auro-dropdown when auro-select is placed inside auro-dialog component.

Please see below videos for related fixes:

Screen.Recording.2024-05-30.at.11.48.46.mov

Also, the fixes seems to not creating any regression when implemented on its own in the docsite:

Screen.Recording.2024-05-30.at.14.21.10.mov

Type of change:

Please delete options that are not relevant.

  • New capability
  • Revision of an existing capability
  • Infrastructure change (automation, etc.)
  • Other (please elaborate)

Checklist:

  • My update follows the CONTRIBUTING guidelines of this project
  • I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.

Thank you for your submission!

-- Auro Design System Team

- This commit addresses the positioning issue of `auro-dropdown`
when `auro-select` is placed inside `auro-dialog` component

Changes to be committed:
	modified:   package-lock.json
	modified:   src/auro-select.js
	modified:   src/style.scss
@fajar-apri-alaska fajar-apri-alaska self-assigned this May 30, 2024
@fajar-apri-alaska fajar-apri-alaska requested a review from a team as a code owner May 30, 2024 07:26
@fajar-apri-alaska fajar-apri-alaska linked an issue May 30, 2024 that may be closed by this pull request
Changes to be committed:
modified:   package-lock.json
modified:   package.json
@blackfalcon blackfalcon force-pushed the fajarapri/fixPositioning branch from ea0591a to 4b43b22 Compare May 30, 2024 16:17
This commit updates the pakcage.json build scripts to be compliant with
the decisions made in this discussion:

https://github.com/orgs/AlaskaAirlines/discussions/523

Changes to be committed:
modified:   .github/workflows/testPublish.yml
modified:   package.json
Copy link
Member

@blackfalcon blackfalcon left a comment

Choose a reason for hiding this comment

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

Changes appear to be addressing the issue and do not have adverse side-effects on UIs not contained within a dialog element.

Copy link
Member

@jason-capsule42 jason-capsule42 left a comment

Choose a reason for hiding this comment

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

@fajar-apri-alaska

I'm struggling to validate that this fix is actually correct.

I am unable to setup a working example that is broken without the style changes and works correctly with them.

The change should include some sort of example that demonstrates this now works. If you add an example to the docpage then we should be able to run this locally and simply comment out the style changes and see it broken.

@fajar-apri-alaska
Copy link
Contributor Author

fajar-apri-alaska commented May 31, 2024

@fajar-apri-alaska

I'm struggling to validate that this fix is actually correct.

I am unable to setup a working example that is broken without the style changes and works correctly with them.

The change should include some sort of example that demonstrates this now works. If you add an example to the docpage then we should be able to run this locally and simply comment out the style changes and see it broken.

@jason-capsule42, Is it really needed to put the example in the docsite?

To be honest, I was also struggling to replicate the issue in the docsite,
before below PR comes from the reporter itself to help reproduce it:
https://github.com/Alaska-ECommerce/LoungeMembership-Web/pull/386

@jason-capsule42 jason-capsule42 self-requested a review June 10, 2024 16:17
@jason-capsule42 jason-capsule42 merged commit 05a25d9 into main Jun 10, 2024
6 checks passed
@jason-capsule42 jason-capsule42 deleted the fajarapri/fixPositioning branch June 10, 2024 16:31
@blackfalcon
Copy link
Member

🎉 This PR is included in version 2.10.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@blackfalcon blackfalcon added the released Completed work has been released label Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Completed work has been released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auro Select's Dropdown displays wonky inside Auro Dialog
3 participants