Posted March 11, 20196 yr comment_5881654 Hi! I've popped back casually after a few years away and noticed the upgrade. One thing isn't quite right though. On mobile, images break the natural page width: It seems to me that the cause is the addition of .cPost div[data-role="commentContent"] img.ipsImage_thumbnailed{max-width:1000px;max-height:750px;} to your framework.css. That's saying that if your image is larger than 1000px wide, then reduce it to that width, which is what is happening. However, it's taking priority over another rule which says do not allow an image to be wider than the page width. I don't know what this rule was added for but it's clearly problematic. (And not the correct approach to take; changes should be made in custom.css. The stock CSS files will either be overwritten in future upgrades (meaning that you lose your changes) or not overwritten (meaning that changes that are part of upgrades won't be introduced) -- the jury's out as to which of the two approaches apply here!) You need to get rid of this rule as it stands because of the knock-on problems it's causing but there'll be an alternative way of doing what you intended. What is it you were trying to achieve by adding this rule?
April 13, 20196 yr comment_5885250 Hey, I'm looking into this and trying to get it resolved. Sorry for the delay. What does this actually look like on your phone? Do you have a screenshot? This was a default setting that happened with the upgrade, not actually something we manually changed.
April 24, 20196 yr Author comment_5886309 On 4/13/2019 at 10:43 PM, Charles (Loss) said: Do you have a screenshot? To read, it's necessary to scroll horizontally, so this happens: On 4/13/2019 at 10:43 PM, Charles (Loss) said: This was a default setting that happened with the upgrade, not actually something we manually changed. You'll be in the clear whenever you upgrade, then; there's no sign of that line in the latest versions' framework.css
Create an account or sign in to comment