Opened 8 years ago

Closed 8 years ago

#2184 closed Bug (fixed)

Lots of html validation errors in the filemanager

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6.1
Component: File Browser Version: FCKeditor 2.6
Keywords: Review+ Cc:

Description

I don't know why we didn't realize earlier, but the filemanager has lots of validations errors that can be solved (most of them) easily.

Attachments (1)

2184.patch (9.8 KB) - added by alfonsoml 8 years ago.
Proposed SVN patch

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by alfonsoml

Proposed SVN patch

comment:1 Changed 8 years ago by alfonsoml

  • Keywords Review? added

The patch fixes most of the problems, except some warnings about framespacing, bordercolor and frameborder but they might be trickier to do them properly as the behavior of styles in the framesets sometimes isn't able to fully reproduce the same effects.

So I suggest to fix the main issues and then if someone wants to play with the frames try it, but meanwhile the rest of issues have been fixed.

comment:2 Changed 8 years ago by fredck

  • Keywords Review+ added; Review? removed

I would just change the "table.fullHeight" selector to ".fullHeight", just to make it more useful... it could be used elsewhere if needed.

We can leave with the framesets warnings for now, until we work on a File Browser review.

Fell free to make or not that CSS change and commit it directly.

comment:3 Changed 8 years ago by fredck

  • Milestone set to FCKeditor 2.6.1

comment:4 Changed 8 years ago by alfonsoml

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [1985]

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy