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

Reduce DOIT logo and include mobile sizing #293

Merged
merged 2 commits into from
Nov 30, 2023
Merged

Reduce DOIT logo and include mobile sizing #293

merged 2 commits into from
Nov 30, 2023

Conversation

jwade1327
Copy link
Member

@jwade1327 jwade1327 commented Nov 29, 2023 •
edited
Loading

Updated screenshot:
Screen Shot 2023-11-30 at 7 29 08 AM

Screen Shot 2023-11-29 at 12 17 28 PM

@jwade1327
Copy link
Member Author

Logo looks blurry on DOIT page

Shinsina
Shinsina reviewed Nov 29, 2023
View reviewed changes
Copy link
Member

@Shinsina Shinsina left a comment

Choose a reason for hiding this comment

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

image

Actually image seems to be 440px wide, the wrapping container is 700 pixels wide, the width on the image itself seems to be 400px so it is still being stretched, not sure what exactly we're trying to do here.

@jwade1327 jwade1327 requested a review from Shinsina November 30, 2023 13:29
@Shinsina Shinsina merged commit 482c096 into parameter1:master Nov 30, 2023
9 checks passed
@jwade1327
Copy link
Member Author

Add Podcast section to CC newsletter

@jwade1327 jwade1327 deleted the doit branch January 30, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shinsina Shinsina Shinsina approved these changes

@B77Mills B77Mills Awaiting requested review from B77Mills

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jwade1327 @Shinsina