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

🎨 Css-ify GitHub #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

🎨 Css-ify GitHub #37

wants to merge 1 commit into from

Conversation

dawsbot
Copy link

@dawsbot dawsbot commented Mar 11, 2017

Works ⚡️-fast, I never even saw a flash of the profile picture. visibility: hidden allows the content to take up the same space as before. This leaves the GitHub UI intact, with white blanks where old content would have been instead of compressing that space.

Examples:
screen shot 2017-03-11 at 3 37 41 am

screen shot 2017-03-11 at 3 42 22 am

If this is good, I'll apply these changes to Twitter as well. Pretty stoked on this, it makes the extension usable for its original purpose without having to make yourself ignore the pictures on initial page load!

* Part of #35
@chrisbodhi
Copy link

🙌 Thanks for picking this up! I've been slammed with work lately, so haven't had the time to revisit it.

@chrisbodhi chrisbodhi mentioned this pull request Mar 13, 2017
@dawsbot
Copy link
Author

dawsbot commented Mar 18, 2017

Thoughts on merging this? I'm not sure what the structure is for owners of this repo. Are you the lead maintainer @chrisbodhi?

@chrisbodhi
Copy link

@dawsbot Nope, I'm just another concerned citizen. For what it's worth, the code looks good to me.

@dawsbot
Copy link
Author

dawsbot commented Mar 28, 2017

@fureigh bump? Would you consider adding @chrisbodhi and I as contributors? It might help take the burdon off you.

@dawsbot
Copy link
Author

dawsbot commented Mar 31, 2017

@fureigh Bump ^

@dawsbot
Copy link
Author

dawsbot commented Jun 19, 2017

Bump @fureigh

@dawsbot
Copy link
Author

dawsbot commented Sep 25, 2017

@fureigh I'm going to fork this project and create a second copy published to the Chrome store if there is no activity here in the next 3 days.

@dawsbot
Copy link
Author

dawsbot commented Apr 18, 2019

@fureigh Any chance you can transfer ownership of this? I'll keep this project alive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants