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

Weird HTML/CSS issue in pastebin #114

Closed
anlutro opened this issue Mar 7, 2014 · 15 comments
Closed

Weird HTML/CSS issue in pastebin #114

anlutro opened this issue Mar 7, 2014 · 15 comments
Labels

Comments

@anlutro
Copy link
Contributor

anlutro commented Mar 7, 2014

See image: http://i.imgur.com/eRkcCLe.png - Notice the scroll bar at the bottom. I get this problem in Chrome (Windows and Linux), FireFox (only tested Windows) and Internet Explorer 11.

This seems to be related to the css of body.bin .show-container { width: 200%; overflow-x: scroll; }

If I remove either the 200% width or the overflow-x things immediately look better. To get rid of the scrollbar at the bottom I need to do overflow-x: hidden on body, which I'm sure is not the intent.

These changes were introduced in #105 and I'd love to hear more reasoning behind them. Were they even tested?

@driesvints
Copy link
Member

Is this still an ongoing issue? I can't seem to reproduce this.

The layout for pastebin has gone under significant changes since it was released so I don't think this still is an issue. I'm gonna close this for now but please let me know here if you're still experiencing this issue.

@anlutro
Copy link
Contributor Author

anlutro commented May 18, 2014

Yes, it's still present exactly as before.

@driesvints
Copy link
Member

Ok. Reopening this. Can you perhaps create a paste where the problem occurs and link it to here in a comment?

@driesvints driesvints reopened this May 18, 2014
@anlutro
Copy link
Contributor Author

anlutro commented May 18, 2014

Looking through random ones in my history, this one: http://laravel.io/bin/jjDql

@driesvints driesvints added the bug label May 20, 2014
@driesvints driesvints added this to the 2.0 milestone May 20, 2014
@driesvints
Copy link
Member

Ok, I'm gonna assign this to @mitchellvanw since he created the pastebin.

@mitchellvanw
Copy link
Contributor

What I'll do for now is remove the width: 200%.

@driesvints
Copy link
Member

This seems fixed now. Can you confirm @anlutro?

@anlutro
Copy link
Contributor Author

anlutro commented Jun 15, 2014

Horizontal scrollbar is gone but there's still a grey bar. http://i.imgur.com/PYPmhqS.png

@anlutro
Copy link
Contributor Author

anlutro commented Jun 16, 2014

The grey bar is a disabled scrollbar. Was kinda hard to see on my linux setup.

@driesvints driesvints modified the milestones: 2.0, 2.1 Aug 17, 2014
@driesvints
Copy link
Member

There have been some changes to the pastebin lately. Is this still an ongoing issue?

If so please post:

  • A link to an example paste that re-creates the bug
  • Tell me which OS and browser version you're using
  • The dimensions of your window when browsing the webpage

Thanks!

@anlutro
Copy link
Contributor Author

anlutro commented Sep 18, 2014

Sorry for the delayed response. Bin link: http://laravel.io/bin/48err

Looks like this on Windows 7/Chrome in ANY window dimension: http://i.imgur.com/IOprTja.png

Looks the same on Debian/Chromium. My previous comment's picture still holds true: http://i.imgur.com/PYPmhqS.png

@driesvints
Copy link
Member

Ok thanks. I'm gonna do some work on the Pastebin this weekend to fix all of the open issues. Will definitely also take a look at this.

@driesvints
Copy link
Member

This should now be fixed in the latest release. Can you confirm?

@anlutro
Copy link
Contributor Author

anlutro commented Sep 21, 2014

Confirmed :)

@driesvints
Copy link
Member

\o/ :smile:

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

No branches or pull requests

3 participants