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

Chore/organization people UI changes #2358

Merged
Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
f3275a5
changed color scheme for the organization people screen
AVtheking Oct 27, 2024
6c57446
fix precommit
AVtheking Oct 27, 2024
6f9b7ac
merge
AVtheking Oct 27, 2024
345703f
merged
AVtheking Oct 27, 2024
dd6ddbe
Update pre-commit
AVtheking Oct 28, 2024
91fc091
fix conflicts
AVtheking Oct 28, 2024
07bf21d
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Oct 28, 2024
ff6d629
fix type checks
AVtheking Oct 28, 2024
7867e6a
fix type checks
AVtheking Oct 28, 2024
4af1540
fix type checks
AVtheking Oct 28, 2024
8bbff3d
fix ts eslint errors
AVtheking Oct 30, 2024
56f857f
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Oct 30, 2024
849ad47
fix ts eslint errors
AVtheking Oct 30, 2024
f13dfad
fix ts eslint errors
AVtheking Oct 30, 2024
3083c4d
fix ts eslint errors
AVtheking Oct 30, 2024
4ed66a5
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Oct 30, 2024
9bf0bf7
testing
AVtheking Oct 30, 2024
07406bb
testing
AVtheking Oct 30, 2024
695c22d
testing
AVtheking Oct 30, 2024
c0ca9c2
reverted changes in yaml file
AVtheking Oct 30, 2024
f2aa7c1
cr comments
AVtheking Oct 31, 2024
80dab7d
Update pull-request.yml
AVtheking Oct 31, 2024
8983ea9
cr comments
AVtheking Oct 31, 2024
7923515
cr comments and single css file
AVtheking Oct 31, 2024
fcb9a9d
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Oct 31, 2024
3ed1122
CR comments
AVtheking Oct 31, 2024
e54e33c
delete button margin from top
AVtheking Oct 31, 2024
a59ce86
prettier for commit and pull request
AVtheking Oct 31, 2024
700d91f
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Oct 31, 2024
6280b0c
remove hard coded colors
AVtheking Nov 5, 2024
200b2af
Merge branch 'develop' into chore/Organization-people-ui-changes
AVtheking Nov 6, 2024
ff383f2
fix failing test cases
AVtheking Nov 6, 2024
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
2 changes: 1 addition & 1 deletion .github/workflows/pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
- name: Get changed TypeScript files
id: changed-files
uses: tj-actions/changed-files@v40

AVtheking marked this conversation as resolved.
Show resolved Hide resolved
- name: Check formatting
if: steps.changed-files.outputs.only_changed != 'true'
run: npm run format:check
Expand Down
6 changes: 3 additions & 3 deletions src/assets/css/app.css

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion src/components/LeftDrawer/LeftDrawer.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
display: flex;
flex-direction: column;
padding: 1rem 1rem 0 1rem;
background-color: var(--bs-white);
background-color: #f6f8fc;
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
transition: 0.5s;
font-family: var(--bs-leftDrawer-font-family);
}
Expand Down
6 changes: 3 additions & 3 deletions src/components/LeftDrawerOrg/LeftDrawerOrg.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
display: flex;
flex-direction: column;
padding: 0.8rem 0rem 0 1rem;
background-color: var(--bs-white);
background-color: #f6f8fc;
transition: 0.5s;
font-family: var(--bs-leftDrawer-font-family);
}
Expand Down Expand Up @@ -105,7 +105,7 @@
}

.leftDrawer .organizationContainer .profileContainer {
background-color: #31bb6b33;
background-color: #e0e9ff;
padding-right: 10px;
}

Expand Down Expand Up @@ -155,7 +155,7 @@
.leftDrawer .profileContainer .profileText .secondaryText {
font-size: 0.8rem;
font-weight: 400;
color: var(--bs-secondary);
/* color: var(--bs-secondary); */
display: block;
text-transform: capitalize;
}
Expand Down
7 changes: 5 additions & 2 deletions src/components/LeftDrawerOrg/LeftDrawerOrg.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -151,16 +151,19 @@ const leftDrawerOrg = ({
<Button
key={name}
variant={isActive === true ? 'success' : ''}
style={{
backgroundColor: isActive === true ? '#EAEBEF' : '',
}}
className={`${
isActive === true ? 'text-white' : 'text-secondary'
isActive === true ? 'text-black' : 'text-secondary'
}`}
>
<div className={styles.iconWrapper}>
<IconComponent
name={name == 'Membership Requests' ? 'Requests' : name}
fill={
isActive === true
? 'var(--bs-white)'
? 'var(--bs-black)'
: 'var(--bs-secondary)'
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
}
/>
Expand Down
44 changes: 38 additions & 6 deletions src/screens/OrganizationPeople/AddMember.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useLazyQuery, useMutation, useQuery } from '@apollo/client';
import { Search } from '@mui/icons-material';
import { Check, Close, Search } from '@mui/icons-material';
import EmailOutlinedIcon from '@mui/icons-material/EmailOutlined';
import Paper from '@mui/material/Paper';
import Table from '@mui/material/Table';
Expand Down Expand Up @@ -33,10 +33,10 @@ import type {
import styles from './OrganizationPeople.module.css';
import Avatar from 'components/Avatar/Avatar';

const StyledTableCell = styled(TableCell)(({ theme }) => ({
const StyledTableCell = styled(TableCell)(() => ({
[`&.${tableCellClasses.head}`]: {
backgroundColor: ['#31bb6b', '!important'],
color: theme.palette.common.white,
backgroundColor: ['#EAEBEF', '!important'],
color: '#555555',
},
[`&.${tableCellClasses.body}`]: {
fontSize: 14,
Expand Down Expand Up @@ -357,8 +357,13 @@ function AddMember(): JSX.Element {
type="submit"
data-testid="submitBtn"
className={`position-absolute z-10 bottom-10 end-0 d-flex justify-content-center align-items-center `}
style={{
marginBottom: '10px',
backgroundColor: '#A8C7FA',
border: '1px #555555 solid',
}}
>
<Search />
<Search style={{ color: '#555555' }} />
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
</Button>
</Form>
</div>
Expand Down Expand Up @@ -413,7 +418,7 @@ function AddMember(): JSX.Element {
</StyledTableCell>
<StyledTableCell align="center">
<Link
className={styles.membername}
className={`${styles.membername} ${styles.subtleBlueGrey}`}
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
to={{
pathname: `/member/${currentUrl}`,
}}
Expand All @@ -431,6 +436,10 @@ function AddMember(): JSX.Element {
createMember(userDetails.user._id);
}}
data-testid="addBtn"
style={{
backgroundColor: '#A8C7FA',
border: '#555555 solid 1px',
}}
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
>
Add
</Button>
Expand Down Expand Up @@ -559,15 +568,38 @@ function AddMember(): JSX.Element {
variant="danger"
onClick={closeCreateNewUserModal}
data-testid="closeBtn"
style={{
backgroundColor: '#F8D6DC',
color: '#555555',
}}
>
<Close
style={{
color: '#FF4D4F',
marginRight: '5px',
}}
/>

AVtheking marked this conversation as resolved.
Show resolved Hide resolved
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
{translateOrgPeople('cancel')}
</Button>
<Button
className={`${styles.colorPrimary} ${styles.borderNone}`}
variant="success"
onClick={handleCreateUser}
data-testid="createBtn"
style={{
backgroundColor: '#A8C7FA',
border: '#555555 solid 1px',
marginLeft: '10px',
color: '#555555',
}}
>
<Check
style={{
color: '#555555', // Green color for the tick icon
marginRight: '5px',
}}
/>
{translateOrgPeople('create')}
</Button>
</div>
Expand Down
58 changes: 48 additions & 10 deletions src/screens/OrganizationPeople/OrganizationPeople.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,57 @@
width: 670px;
max-width: 680px;
}

.subtleBlueGrey {
color: #7c9beb;
text-decoration: none;
}

.subtleBlueGrey:hover {
color: #5f7e91;
text-decoration: underline;
}
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
.dropdown {
background-color: white;
border: 1px solid #31bb6b;
border: 1px solid #555555;
position: relative;
display: inline-block;
margin-top: 10px;
margin-bottom: 10px;
color: #31bb6b;
color: #555555;
}

.dropdown:hover,
.dropdown:focus,
.dropdown:active .dropdown:focus-visible .dropdown.show {
background-color: transparent !important;
border: 1px solid #555555;
color: #555555 !important;
}
.dropdown:focus,
.dropdown:focus-visible,
.dropdown:active .dropdown.show {
background-color: white !important;
color: black !important;
outline: none !important;
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
}

.dropdownItem {
background-color: white !important;
color: #555555 !important;
border: none !important;
}

/* Override Bootstrap's hover, focus, and active styles for dropdown items */
.dropdownItem:hover,
.dropdownItem:focus,
.dropdownItem:active {
background-color: white !important; /* Keep the background white */
color: #555555 !important; /* Keep the text color as default */
outline: none !important;
box-shadow: none !important; /* Remove any shadow */
}
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
AVtheking marked this conversation as resolved.
Show resolved Hide resolved

.input {
flex: 1;
position: relative;
Expand All @@ -42,10 +84,6 @@
position: relative;
}

/* input {
outline: 1px solid var(--bs-gray-400);
} */

.btnsContainer .input button {
width: 52px;
}
Expand All @@ -54,25 +92,25 @@
margin-top: 10px;
margin-bottom: 10px;
background-color: white;
box-shadow: 0 1px 1px #31bb6b;
box-shadow: 0 1px 1px #dddddd;
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
}
.inputFieldModal {
margin-bottom: 10px;
background-color: white;
box-shadow: 0 1px 1px #31bb6b;
box-shadow: 0 1px 1px #dddddd;
}
.inputField > button {
padding-top: 10px;
padding-bottom: 10px;
}
.TableImage {
object-fit: cover;
margin-top: 15px !important;
width: 50px !important;
height: 50px !important;
border-radius: 100% !important;
}
.tableHead {
background-color: #31bb6b !important;
color: white;
border-radius: 20px !important;
padding: 20px;
Expand All @@ -95,7 +133,7 @@
}
.tableHeader {
background-color: var(--bs-primary);
color: var(--bs-white);
color: var(--bs-greyish-black);
AVtheking marked this conversation as resolved.
Show resolved Hide resolved
font-size: 16px;
}

Expand Down
Loading
Loading