[bug like report] forum/blog etc image size issues

soaringeagle
@soaringeagle
9 years ago
3,304 posts
posts like this
https://www.dreadlockssite.com/dreadlocks-forums/forum/new_posts/56967/the-beauty-of-the-early-messy-stage-of-natural-neglect-dreadlocks
had massive images, there was a ?width=799 & height=555 type stuff attached to each url even after stripping them the images still cause the page to be a disaster

what can be done either through css or better yet the image resizer (as well as code stripper i guess) to fix this

iknow its not technicaly a bug just a diference in how ning and jr handle things

keep in mind i need a post import solution even if u work out a during import 1


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities

updated by @soaringeagle: 05/18/15 05:18:48PM
michael
@michael
9 years ago
7,715 posts
Those images are not going through the image processing system, so there's not much your Jamroom can do to take control of them.

You could put a CSS limiter in there to bring them down to size.
.forum_post_text img{
  max-width:500px;
}
should do it.

That's not going to help the processing time though as each image will still be massive. A better solution would be to get those images being passed through Jamroom's image functions to resize and cache them.

I'll take a look and see if that's possible.
soaringeagle
@soaringeagle
9 years ago
3,304 posts
yea that will only help with pc view too not mobile


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
michael
@michael
9 years ago
7,715 posts
You could do the same inside the mobile view to limit it to less. There should be a mobile specific .css sheet for most skins. Would that work?

If it wasn't done via CSS how wide would you like it? 100%?
soaringeagle
@soaringeagle
9 years ago
3,304 posts
probably

though i thought i tried that


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
soaringeagle
@soaringeagle
9 years ago
3,304 posts
but then again i dont think % values overwrite px values coded into the url
that maybe was the issue it was 100% of the 779 px or whatever
i actualy went to max width 24% to get it to fit i think


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
soaringeagle
@soaringeagle
9 years ago
3,304 posts
yea tested only px values are going to work


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
michael
@michael
9 years ago
7,715 posts
in jrElastic there is a css file:
/skins/jrElastic/css/override_mobile.css

Which has mobile specific CSS. Could you do the same with your skin? Put one default px width in for mobiles and another in for non-mobile perhaps.
soaringeagle
@soaringeagle
9 years ago
3,304 posts
mobile cores the same as override mobile i assumed right


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
douglas
@douglas
9 years ago
2,790 posts
soaringeagle:
mobile cores the same as override mobile i assumed right

Yes, that is correct.


--

Douglas Hackney
Jamroom Team - Designer/Developer/Support
FAQ-Docs-Help Videos
soaringeagle
@soaringeagle
9 years ago
3,304 posts
i dont own a mobile to test bur would 200 px be sufficeint or what would u suggest


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
douglas
@douglas
9 years ago
2,790 posts
You could probably go as wide as 290px and it should be okay on smart phones.


--

Douglas Hackney
Jamroom Team - Designer/Developer/Support
FAQ-Docs-Help Videos
soaringeagle
@soaringeagle
9 years ago
3,304 posts
some of the text was wrapped round the image instead of below
so maybe 200 is a good compromise


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities
douglas
@douglas
9 years ago
2,790 posts
soaringeagle:
some of the text was wrapped round the image instead of below
so maybe 200 is a good compromise

If you have wrapped text, you might go a little lower, say 128px


--

Douglas Hackney
Jamroom Team - Designer/Developer/Support
FAQ-Docs-Help Videos
soaringeagle
@soaringeagle
9 years ago
3,304 posts
thnx


--
soaringeagle
head dreadhead at dreadlocks site
glider pilot student and member/volunteer coordinator with freedoms wings international soaring for people with disabilities

Tags