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

Jump to bottom

[EC-641] Browser Ext UI Update #3842

Merged
merged 27 commits into from
Nov 1, 2022
Merged

[EC-641] Browser Ext UI Update #3842

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
cd8cc63
more css changes
DanHillesheim Oct 7, 2022
d57e909
add icon button hover
DanHillesheim Oct 19, 2022
fd1f0b6
Update apps/browser/src/popup/scss/box.scss
DanHillesheim Oct 20, 2022
1d42dbe
Update apps/desktop/src/scss/box.scss
DanHillesheim Oct 20, 2022
1259cd0
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 20, 2022
4d4b10f
feedback updates
DanHillesheim Oct 20, 2022
ad4eb68
Merge branch 'browser-ext-ui-update-test' of https://github.com/bitwa…
DanHillesheim Oct 20, 2022
623240e
restore desktop pseudo rule
DanHillesheim Oct 24, 2022
f645129
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 24, 2022
e271198
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 26, 2022
b5c7a10
update to include some variable fixes and deletions
DanHillesheim Oct 26, 2022
4004219
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 26, 2022
be1b412
updates per oscar
DanHillesheim Oct 27, 2022
3a3b667
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 27, 2022
31bf01f
feedback updates
DanHillesheim Oct 27, 2022
51615c5
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 27, 2022
8bd18f5
changes per product design
DanHillesheim Oct 28, 2022
fbc3adc
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 28, 2022
0aa4103
add more helper text space below for visual separation
DanHillesheim Oct 28, 2022
de15d9b
group new variable
DanHillesheim Oct 28, 2022
246768c
login page button fix
DanHillesheim Oct 28, 2022
b800710
Merge branch 'master' into browser-ext-ui-update-test
DanHillesheim Oct 31, 2022
ae661aa
Revert "Merge branch 'master' into browser-ext-ui-update-test"
DanHillesheim Oct 31, 2022
8f2ae2b
fix button height
DanHillesheim Oct 31, 2022
a0ef655
Merge branch 'master' into browser-ext-ui-update-test
kspearrin Nov 1, 2022
46eafaa
revert file changes
kspearrin Nov 1, 2022
03c95b5
test adjustments
kspearrin Nov 1, 2022
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
53 changes: 34 additions & 19 deletions apps/browser/src/popup/scss/box.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
.box {
position: relative;
width: 100%;
margin: 10px 0;
margin: 10px 0 20px;
Hinton marked this conversation as resolved.
Show resolved Hide resolved

&.first {
margin-top: 0;
Expand Down Expand Up @@ -36,7 +36,7 @@
&:focus-visible,
&.active {
@include themify($themes) {
background-color: themed("boxBackgroundHoverColor");
background-color: themed("backgroundColor");
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
}
}

Expand All @@ -52,11 +52,11 @@
}

.box-content {
border-top: 1px solid #000000;
border-bottom: 1px solid #000000;
Copy link
Member

Choose a reason for hiding this comment

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

Since we removed the borders from the boxes, Solarize Dark theme seems to need adjustments as there is nothing visually separating the rows.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I addressed the issue for the most part. I think there's a little more cleanup to do possibly.

Copy link
Member

@danielleflinn danielleflinn Oct 20, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Looks good! Only bit of feedback is we should probably add a new lighter variable for the hover color so that the hover state is retained.

Edit: actually hover state in settings looks good; however it isn't showing up on the main Tab or Vault pages or 'password history' in Generator

// border-top: 1px solid #000000;
// border-bottom: 1px solid #000000;
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved

@include themify($themes) {
background-color: themed("boxBackgroundColor");
background-color: themed("backgroundColor");
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
border-color: themed("borderColor");
}

Expand Down Expand Up @@ -91,11 +91,15 @@
&.list {
.box-content {
.box-content-row {
padding: 3px 10px;
padding: 10px;
text-decoration: none;
border-radius: 6px;
margin: 10px;
background-color: $background-color;
Hinton marked this conversation as resolved.
Show resolved Hide resolved

@include themify($themes) {
color: themed("textColor");
background-color: themed("boxBackgroundColor");
}

&.padded {
Expand Down Expand Up @@ -150,6 +154,8 @@
display: flex;
padding-top: 10px;
padding-bottom: 10px;
margin: 10px;
border-radius: 6px;
}
}
}
Expand Down Expand Up @@ -191,21 +197,28 @@
padding: 10px 15px;
position: relative;
z-index: 1;
background-color: $background-color;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
border-radius: 6px;
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
margin: 10px;

&:before {
content: "";
position: absolute;
right: 0;
bottom: 0;
height: 1px;
width: calc(100% - 10px);
border-bottom: 1px solid #000000;

@include themify($themes) {
border-bottom-color: themed("boxBorderColor");
}
@include themify($themes) {
background-color: themed("boxBackgroundColor");
}

// &:before {
// content: "";
// position: absolute;
// right: 0;
// bottom: 0;
// height: 1px;
// width: calc(100% - 10px);
// border-bottom: 1px solid #000000;

// @include themify($themes) {
// border-bottom-color: themed("boxBorderColor");
// }
// }

Hinton marked this conversation as resolved.
Show resolved Hide resolved
&:last-child {
&:before {
border: none;
Expand Down Expand Up @@ -449,9 +462,11 @@
select {
width: 100%;
border: 1px solid #000000;
border-radius: $border-radius;
border-radius: 6px;
padding: 7px 4px;

@include themify($themes) {
background-color: themed("boxBorderColor");
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
border-color: themed("inputBorderColor");
}
}
Expand Down
8 changes: 7 additions & 1 deletion apps/browser/src/popup/scss/buttons.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,8 +89,14 @@

button.box-content-row {
display: block;
width: 100%;
// width: 100%;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
width: calc(100% - 20px);
text-align: left;

@include themify($themes) {
background-color: themed("boxBackgroundColor");
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
border-color: none;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
}
}

button {
Expand Down
34 changes: 31 additions & 3 deletions apps/browser/src/popup/scss/pages.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,26 +9,54 @@ app-sync {
}

app-generator .generated-block {
font-size: $font-size-large;
font-size: $font-size-xlarge;
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
font-family: $font-family-monospace;
margin: 20px;
margin: 10px;
padding: 10px;
display: flex;
background-color: $box-background-color;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
border-radius: 6px;

@include themify($themes) {
background-color: themed("boxBackgroundColor");
}

.generated-wrapper {
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
text-align: left;
width: 100%;
min-width: 0;
white-space: pre-wrap;
word-break: break-all;
padding: 15px 30px;
border-radius: 6px;
background-color: #e6e9ed;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
border: 1px solid;
border-color: #d2d9e6;

@include themify($themes) {
background-color: themed("backgroundColor");
border-color: themed("boxBorderColor");
}
}

.action-buttons {
display: flex;
flex-direction: column;
align-self: center;
height: 100%;
margin-left: 10px;

button {
padding: 5px;
margin: -5px -5px 5px 5px;
margin: 0;

&:hover {
color: $brand-primary;
Hinton marked this conversation as resolved.
Show resolved Hide resolved
DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved

@include themify($themes) {
color: $icon-hover-color;
}
}
}
}
}
Expand Down
3 changes: 3 additions & 0 deletions apps/browser/src/popup/scss/variables.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ $font-family-sans-serif: "Open Sans", "Helvetica Neue", Helvetica, Arial, sans-s
$font-family-monospace: Menlo, Monaco, Consolas, "Courier New", monospace;
$font-size-base: 14px;
$font-size-large: 18px;
$font-size-xlarge: 22px;
$font-size-xxlarge: 28px;
$font-size-small: 12px;
$text-color: #000000;
$border-color: #f0f0f0;
Expand All @@ -15,6 +17,7 @@ $list-icon-color: #767679;
$disabled-box-opacity: 1;
$border-radius: 3px;
$line-height-base: 1.42857143;
$icon-hover-color: lighten($text-color, 50%);

$gray: #555;
$gray-light: #777;
Expand Down
26 changes: 13 additions & 13 deletions apps/desktop/src/scss/box.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,19 +120,19 @@
z-index: 1;
text-align: left;

&:before {
content: "";
position: absolute;
right: 0;
bottom: 0;
height: 1px;
width: calc(100% - 10px);
border-bottom: 1px solid #000000;

Hinton marked this conversation as resolved.
Show resolved Hide resolved
@include themify($themes) {
border-bottom-color: themed("boxBorderColor");
}
}
// &:before {
// content: "";
// position: absolute;
// right: 0;
// bottom: 0;
// height: 1px;
// width: calc(100% - 10px);
// border-bottom: 1px solid #000000;

// @include themify($themes) {
// border-bottom-color: themed("boxBorderColor");
// }
// }

DanHillesheim marked this conversation as resolved.
Show resolved Hide resolved
&:first-child,
&:last-child {
Expand Down