Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Tweak styles #649

Merged
merged 11 commits into from
Oct 16, 2023
Merged

Tweak styles #649

merged 11 commits into from
Oct 16, 2023

Conversation

richtabor
Copy link
Member

@richtabor richtabor commented Oct 13, 2023

Adds more gradients/duotones, and makes ice more unique.

Props @beafialho.

CleanShot 2023-10-13 at 17 27 53
CleanShot 2023-10-13 at 17 28 03
CleanShot 2023-10-13 at 17 28 11
CleanShot 2023-10-13 at 17 28 21

@richtabor richtabor marked this pull request as ready for review October 13, 2023 21:29
styles/ice.json Outdated
},
"layout": {
"contentSize": "620px",
"wideSize": "99vw"
Copy link
Collaborator

Choose a reason for hiding this comment

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

this makes wide width content virtually full width. I suppose that's the intent, just pointing it out

Copy link
Collaborator

Choose a reason for hiding this comment

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

The home looks a little weird on really big screens

Copy link
Collaborator

Choose a reason for hiding this comment

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

Screenshot 2023-10-16 at 12 05 53

Copy link
Collaborator

@MaggieCabrera MaggieCabrera left a comment

Choose a reason for hiding this comment

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

my comments are not blockers, generally these look good to me

styles/rust.json Outdated
@@ -47,49 +57,6 @@
"layout": {
"contentSize": "620px",
"wideSize": "90vw"
Copy link
Member

Choose a reason for hiding this comment

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

As Maggie mentioned above for the Ice style variation, I noticed the same with Rust too - however it was already here before. I find this a bit unlucky, as on larger screens and especially curved monitors with a wide width, these layouts would look pretty much out of place.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll remove it on rust too.

@richtabor
Copy link
Member Author

Updated the styles to this:

CleanShot.2023-10-16.at.12.55.03.mp4

@richtabor richtabor merged commit 6d57c4b into trunk Oct 16, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants