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

feature: improve extension in-tab view #988

Merged
merged 3 commits into from
Jun 3, 2024

Conversation

ost-ptk
Copy link
Member

@ost-ptk ost-ptk commented May 15, 2024

Make sure to fill in all the below sections.

Description

  • removed footer absolute button container
  • fixed modal container and overlay

Linked tickets

WALLET-347

Checklist

  • Make sure this PR title follows semantic release conventions: https://semantic-release.gitbook.io/semantic-release/#commit-message-format

  • If the PR adds any new text to the UI, make sure they are localized

  • Include a screenshot or recording if implementing significant UI or user flow change

  • When this PR affects architecture changes wait for review from Dmytro before merging

@ost-ptk ost-ptk requested a review from Comp0te May 15, 2024 08:32
…n-in-tab-view

# Conflicts:
#	src/apps/popup/app-router.tsx
#	src/libs/layout/unlock-vault/index.tsx
The Overlay component's position and size properties were adjusted to fit its new design requirements. The width is now fixed, and it is centered using transform. Meanwhile, explicit delays were added in the end-to-end tests for 'buy-cspr.spec.ts' to ensure proper closing of modal windows before continuing with the tests.
@Comp0te Comp0te merged commit 9119ad7 into develop Jun 3, 2024
5 checks passed
@Comp0te Comp0te deleted the WALLET-347-cw-improve-extension-in-tab-view branch June 3, 2024 21:03
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.

None yet

2 participants