Modify

Opened 9 years ago

Closed 9 years ago

#3771 closed defect (fixed)

Switch from hardcoded HTML formatting to CSS for images

Reported by: izzy Owned by: Radek Bartoň
Priority: normal Component: ScreenshotsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

The border attributes (and some others) are right now rendered as HTML attributes rather than CSS code - this makes it hard to override them by Stylesheets. E.g. the default value for the border attribute is "1", so if not explicitly specified otherwise, it will turn out as <img border="1" ...> in the page, which causes nasty red boxes around. Would be nice if that could be changed to <img style="border: 1px; ...> instead.

This is affecting Trac 0.10 *and* 0.11 versions.

Funny: On the 0.11 installation we tapped into that, it seems that only Screenshots affected when no align was explicitly defined...

Attachments (0)

Change History (5)

comment:1 Changed 9 years ago by Radek Bartoň

Status: newassigned

As I checked now, all borders are specified the second way (<img style="border: 1px; ...>). Could you sent me macro signature wich is generating borders as HTML attributes?

comment:2 Changed 9 years ago by Radek Bartoň

What about this?

comment:3 Changed 9 years ago by Radek Bartoň

OK, r4344 changes behavior of align=center}} attribute so that it displays image as a block element centered on the page and not as inline element in text flow. {{{align=left and align=right diplays image as floating elements on left resp. right side of page. If you can, try it in IE and if you are satisfied close this ticket, please.

comment:4 Changed 9 years ago by izzy

Well, that's a double problem: Guess it's only applied to the 0.11 branch (where I do not have Shell access, in that project I'm a "normal member" - my own Trac installation is still 0.10), as there came no new files on svn up on the 0.10 branch - and second, there is no IE for Linux (well, even if there were, I would not use it).

So it looks like I can not test it before you're done with the other RFEs and have ported them all to the 0.10 branch, sorry...

comment:5 Changed 9 years ago by Radek Bartoň

Resolution: fixed
Status: assignedclosed

This will stay only in 0.11 branch.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Radek Bartoň.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.