-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
refactor(UI): clean up portal pages #29018
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #29018 +/- ##
===========================================
+ Coverage 63.84% 64.12% +0.27%
===========================================
Files 817 817
Lines 58161 58161
===========================================
+ Hits 37133 37294 +161
+ Misses 21028 20867 -161
|
The empty states feel too empty. Perhaps a white card with the illustration in the center would be better ?The sidebar items disappear when i go to any page like
|
@marination both the issues are to be fixed in the framework. We can go ahead with this PR if everything else is fine. |
@sumaiya2908 can you address the failing linter ? |
@marination is there any other test to address for this PR? |
This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing. |
This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing. |
@Mergifyio backport version-14-hotfix |
✅ Backports have been created
|
* refactor: list view for portal pages * refactor: add align center class * refactor: change footer btn class(primary) * refactor: add order style page * refactor: import order page in website * refactor: remove table elements * fix: preview image height * fix: make string translatable * refactor: change font variables * refactor: order preview bg white * refactor: center align items * fix: breadcrumb padding * refactor: make preview image rounded * refactor: add condition for image container * refactor: change alignment for mobile view * fix: make footer button secondary * refactor: clean code * refactor: code indentantion * refactor: remove space * fix: Payment section cleanup Co-authored-by: Summayya <frappe@Summayyas-MacBook-Air.local> Co-authored-by: Marica <maricadsouza221197@gmail.com> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com> (cherry picked from commit da43a5e)
refactor(UI): clean up portal pages (#29018) * refactor: list view for portal pages * refactor: add align center class * refactor: change footer btn class(primary) * refactor: add order style page * refactor: import order page in website * refactor: remove table elements * fix: preview image height * fix: make string translatable * refactor: change font variables * refactor: order preview bg white * refactor: center align items * fix: breadcrumb padding * refactor: make preview image rounded * refactor: add condition for image container * refactor: change alignment for mobile view * fix: make footer button secondary * refactor: clean code * refactor: code indentantion * refactor: remove space * fix: Payment section cleanup Co-authored-by: Summayya <frappe@Summayyas-MacBook-Air.local> Co-authored-by: Marica <maricadsouza221197@gmail.com> Co-authored-by: Deepesh Garg <deepeshgarg6@gmail.com> (cherry picked from commit da43a5e) Co-authored-by: Summayya Hashmani <58825865+sumaiya2908@users.noreply.github.com>
Needs to be merged after this
Before
![Screenshot 2021-12-24 at 12 09 18 PM](https://user-images.githubusercontent.com/58825865/147325780-c201ad7a-85ad-4eb5-b654-04c2057d9c1e.png)
After
![Screenshot 2021-12-24 at 12 08 11 PM](https://user-images.githubusercontent.com/58825865/147325795-a541593c-b03b-45fa-bf1e-d6b01304fc4b.png)
Before
After
Mobile View
Pages Affected