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

soaringeagle
@soaringeagle
10 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
10 years ago
7,778 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
10 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
10 years ago
7,778 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
10 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
10 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
10 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
10 years ago
7,778 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
10 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
10 years ago
2,797 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
10 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
10 years ago
2,797 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
10 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
10 years ago
2,797 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
10 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