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.

Sign up for GitHub

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

Improve multi emails and mobiles field UI #7277

Merged
merged 9 commits into from
Jan 10, 2025
Merged

Improve multi emails and mobiles field UI #7277

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a0d740a
Change primary icon and icon buttons to text buttons
PasinduYeshan Jan 8, 2025
da4ee09
Remove unnecessary accordion
PasinduYeshan Jan 8, 2025
653502c
myaccount-change primary icon and icon buttons
PasinduYeshan Jan 8, 2025
4e685e5
Update css to make rows same size - console
PasinduYeshan Jan 8, 2025
77633d0
Hide select in single value mode
PasinduYeshan Jan 8, 2025
6c727e2
Enable multi emails feature
PasinduYeshan Jan 8, 2025
c1394df
Add changeset
PasinduYeshan Jan 8, 2025
d2e5c5e
Refactor code
PasinduYeshan Jan 9, 2025
2ac5589
Remove spaces
PasinduYeshan Jan 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .changeset/long-rules-joke.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
"@wso2is/admin.users.v1": patch
"@wso2is/myaccount": patch
"@wso2is/console": patch
---

Improve multi email and mobile UI
2 changes: 1 addition & 1 deletion apps/console/src/public/deployment.config.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1282,7 +1282,7 @@
"isHeaderAvatarLabelAllowed": false,
"isLeftNavigationCategorized": true,
"isMarketingConsentBannerEnabled": false,
"isMultipleEmailsAndMobileNumbersEnabled": false,
"isMultipleEmailsAndMobileNumbersEnabled": true,
"isPasswordInputValidationEnabled": true,
"isRequestPathAuthenticationEnabled": false,
"isSAASDeployment": false,
Expand Down
27 changes: 14 additions & 13 deletions apps/myaccount/src/components/profile/profile.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,44 +21,45 @@
width: 100%;
}

.accordion-details {
border-radius: 8px;
padding: 2px;

.multi-value-table {
.multi-value-table-data-row {
&:last-child td, &:last-child th {
border: none;
}
.multi-value-table {
.multi-value-table-data-row {
&:last-child td, &:last-child th {
border: none;
}

.table-c1 {
display: flex;
flex-wrap: nowrap;
align-items: center;

padding: 0.2em 0;

.c1-value{
margin-bottom: 0;
flex-shrink: 0;
padding-right: 12px;
min-width: 30ch;

PasinduYeshan marked this conversation as resolved.
Show resolved Hide resolved
&.mobile-label {
min-width: 20ch;
}
}

PasinduYeshan marked this conversation as resolved.
Show resolved Hide resolved
.verified-icon, .primary-icon {
align-items: center;
}
}

PasinduYeshan marked this conversation as resolved.
Show resolved Hide resolved
.table-c2 {
display: flex;
flex-wrap: nowrap;
align-items: center;
justify-content: flex-end;
gap: 4px;
padding: 0.2em 0;

.text-btn {
padding: 0px;
}
}
}
}
Expand Down
419 changes: 215 additions & 204 deletions apps/myaccount/src/components/profile/profile.tsx
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion apps/myaccount/src/public/deployment.config.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@
},
"isCookieConsentBannerEnabled": true,
"isHeaderAvatarLabelAllowed": true,
"isMultipleEmailsAndMobileNumbersEnabled": false,
"isMultipleEmailsAndMobileNumbersEnabled": true,
"isPasswordInputValidationEnabled": true,
"isProfileUsernameReadonly": true,
"privacyPolicyConfigs": {},
Expand Down
93 changes: 33 additions & 60 deletions features/admin.users.v1/components/user-profile.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,75 +16,48 @@
* under the License.
*/

.multi-valued-accordion {
.accordion-summary {
display: flex;
padding: 0px 12px 0px 8px;
flex-wrap: nowrap;
align-items: center;
gap: 4px;
border-radius: 8px;

.accordion-label {
margin: 0 0 0 8px;
flex-shrink: 0;
padding-right: 4px;
min-width: 250px;

&.mobile-label {
min-width: 150px;
}
.multi-value-table {
.multi-value-table-data-row {
&:last-child td, &:last-child th {
border: none;
}

.verified-icon, .primary-icon {
.table-c1 {
display: flex;
flex-wrap: nowrap;
align-items: center;
padding: 0 4px;
}
}

.accordion-details {
border-radius: 8px;
.multi-value-table {
.multi-value-table-data-row {
&:last-child td, &:last-child th {
border: none;
font-size: 1rem;
padding: 0.2em 0;

.c1-value{
margin-bottom: 0;
flex-shrink: 0;
padding-right: 12px;
min-width: 250px;

&.mobile-label {
min-width: 150px;
}
}

.table-c1 {
display: flex;
flex-wrap: nowrap;

.verified-icon, .primary-icon {
align-items: center;
font-size: 1rem;

.c1-value{
margin-bottom: 0;
flex-shrink: 0;
padding-right: 12px;
min-width: 250px;

&.mobile-label {
min-width: 150px;
}
}

.verified-icon, .primary-icon {
align-items: center;
padding: 0 4px;
}
padding: 0 4px;
}
}

.table-c2 {
display: flex;
flex-wrap: nowrap;
align-items: center;
justify-content: flex-end;
gap: 4px;
padding: 0.2em 0;

.table-c2 {
display: flex;
flex-wrap: nowrap;
align-items: center;
justify-content: flex-end;
gap: 4px;
.text-btn {
padding: 0px;
white-space: nowrap;
}
}
}

.MuiAccordionDetails-root {
padding: 0px;
}
}
Loading
Loading