Opened 13 years ago
Closed 13 years ago
Last modified 13 years ago
#9713 closed Patch - Bug Fix (Fixed)
Fix scaling of channel icons in recorded programs screen
Reported by: | Owned by: | sphery | |
---|---|---|---|
Priority: | minor | Milestone: | 0.25 |
Component: | Plugin - MythWeb | Version: | 0.24-fixes |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
In the last few versions of MythWeb, the default recorded programs screen leveraged a "width" attribute in the <img> tag. This caused channel icons to scale correctly in 95% of cases where the width of the icon was larger or close to the same size as the height. However, in cases where the height of the channel icon was significantly larger than the width, the icon would scale to 70px wide, but still be very tall and distort the height of the row compared to other rows.
Simple HTML attributes in the <img> tag don't really do what we want as it doesn't take into account max scale of width or height. Also, CSS for max-width and max-height aren't properly reflected in IE6 and IE7. Doing some google searching, I found a solution that should work for all modern browsers and IE6/7.
Attached is a patch which removes HTML attribute and changes it to leverage CSS. The patch fixes all current skins (include adding some css files in skins missing them). The patch was made against 0.24-fixes, but I believe it should apply cleanly to master. Tested in IE8, IE9, Chrome 10, and Firefox 4.
Attachments (2)
Change History (6)
Changed 13 years ago by
Attachment: | MythWeb-ChannelIconEdits.diff added |
---|
Changed 13 years ago by
Attachment: | MythWeb-ChannelIconEdits-v2.diff added |
---|
comment:1 Changed 13 years ago by
Uploaded v2 of the patch - minor changes to keep row height for each show consistent regardless of icon height (or lack of icon).
comment:2 Changed 13 years ago by
Status: | new → assigned |
---|
comment:3 Changed 13 years ago by
Milestone: | unknown → 0.25 |
---|---|
Resolution: | → Fixed |
Status: | assigned → closed |
comment:4 Changed 13 years ago by
Owner: | changed from Rob Smith to sphery |
---|
Version 2 - additional tweaks