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

BC-6193 - vue3 fix UI light box #3032

Merged
merged 9 commits into from
Jan 24, 2024

Conversation

SevenWaysDP
Copy link
Contributor

Short Description

Links to Ticket and related Pull-Requests

Changes

Data-security

Deployment

New Repos, NPM packages or vendor scripts

Screenshots of UI changes

Checklist before merging

  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • PO: Any deviation from requirements was agreed with Product-Owner / ticket author / support-team
  • DEV: Every new component is implemented having accessibility in mind (e.g. aria-label, role property)

Notice: Please keep this Pull-Request as a Draft (or add WIP label), until it is ready to be reviewed

@SevenWaysDP SevenWaysDP changed the base branch from main to BC-6196-vue3-tests January 23, 2024 08:09
@SevenWaysDP SevenWaysDP changed the base branch from BC-6196-vue3-tests to BC-3844-vue3-webpack January 23, 2024 08:09
@@ -66,14 +65,8 @@ export default defineComponent({
imageSrc,
setError,
isError,
errorImage,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
errorImage,

@@ -102,8 +102,6 @@ export default defineComponent({
.display-container {
position: relative;
min-height: 52px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Das gehört hier eigentlich auch nicht hin. Willst du das hier noch machen oder in der BC-6355?

@SevenWaysDP SevenWaysDP merged commit ae5beb2 into BC-3844-vue3-webpack Jan 24, 2024
27 of 31 checks passed
@SevenWaysDP SevenWaysDP deleted the BC-6193-vue3-fix-ui-light-box branch January 24, 2024 13:10
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.

2 participants