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

Widen Readmes and clean for comment box index on commit code view #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

MaximusMcCann
Copy link

No description provided.

@tmlee
Copy link
Owner

tmlee commented Jun 4, 2014

@mccannmax thanks for your commits
it looks like some of the div on github has changed that has cause this extension to break.
I could fix that once i merge this in.
What is div.file-box targeting? The file explorer?

@MaximusMcCann
Copy link
Author

When just viewing files on github, say, in the commit history.
image

@tmlee
Copy link
Owner

tmlee commented Jun 5, 2014

@mccannmax I had a problem with some of the comment box appearing on top of the widen code view. I just pushed a new commit applying a new css rule on the #js-repo-pjax-container instead.
Would you give it a try to see if it does what you intended since .file-box falls under #js-repo-pjax-container?

@MaximusMcCann
Copy link
Author

@tmlee had some issues when toggling the plugin and having github not go back to normal. my fork master has what works for me. only point: my way just floats the important code pieces on top to widen it. aka things get hidden behind and are unusable until toggled off. Cheers.

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