Skip to content

removed min/max page width #4

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

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

removed min/max page width #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
fed66f6
removed min/max page width
Sorunome Jun 29, 2015
595128e
Update index.css
Sorunome Jun 29, 2015
bf9a2e1
Update theme.js
Sorunome Jun 29, 2015
d6ce0ff
Update theme.js
Sorunome Jun 29, 2015
27d642c
woops
Sorunome Jun 29, 2015
cab96ef
else lines get tiresome to read
Sorunome Jul 31, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion css/index.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ body, td, th, tr{
div#wrapper{
border-radius: 5px;
margin: 0 auto;
min-width: 1035px;
max-width: 2300px;
min-width: 750px;
}

/* lets give all forms zero padding/margins */
Expand Down
4 changes: 2 additions & 2 deletions scripts/theme.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,14 +95,14 @@ $(function(){
var $sheet = $('<style>').appendTo('head');
Copy link
Member

@Eeems Eeems Jul 10, 2015

Choose a reason for hiding this comment

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

I'm getting an illegal string was specified error when trying to run this section of code manually (Not this specific line)

$(window).resize(function(){
// 90% - 88px - 15em(of .poster)
var w = parseFloat($(window).width())*0.9-88-(15*Number(getComputedStyle($('.poster')[0], "").fontSize.match(/(\d*(\.\d*)?)px/)[1]));
var w = parseFloat($('#wrapper').width())*0.9-88-(15*Number(getComputedStyle($('.poster')[0], "").fontSize.match(/(\d*(\.\d*)?)px/)[1]));
$sheet.html('img.bbc_img{max-width:'+w+'px;}');
}).resize();
}
if($('.dp_news').length > 0){ // front page news
$sheet = $('<style>').appendTo('head');
$(window).resize(function(){
var w = parseFloat($(window).width())*0.9 - 500;
var w = parseFloat($('#wrapper').width())*0.9 - 500;
$sheet.html('img.bbc_img{max-width:'+w+'px;}');
}).resize();
}
Expand Down