Ticket #6465 (closed Bug: fixed)

Opened 5 years ago

Last modified 3 years ago

Plone needs an ID that only surrounds the actual rendered content

Reported by: limi Owned by: limi
Priority: minor Milestone: 3.0b2
Component: Templates/CSS Keywords:
Cc:

Description

Currently, there's a lot of IDs and classes in Plone that try to encapsulate the actual content area (ie. not the widgets around it, but only Title/Description/Body). All of them fail. ;)

The current list includes:

  • #region-content
  • #content
  • .documentContent
  • .documentEditable

Since there is duplication in the #region-content and #content IDs, I propose that we fix it by moving the #content ID to be this wrapper (as a bonus, it's also the shortest of the two). This will make it much easier to do specific skinning for the content area, which is very common (think visited links, for example).

Change History

comment:1 Changed 5 years ago by limi

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

(In [14421]) Cleaned up the ID mess around which ID/class that wraps the actual content (without the UI elements in the content well). The official approach is now to use #content for the actual content, and to use #portal-column-content if you want the entire content column. Also made it possible to get rid of some !important statements. If you have CSS that depends on either of these two selectors, you might want to make sure that the output is still as expected. It should work the same way in most cases, but since the scope is narrower for #content now, you might need to update your styles (and main_template needs to be updated too, if you have customized that). Fixes #6465.

comment:2 Changed 3 years ago by hannosch

  • Component changed from Visual and templates to Templates/CSS
Note: See TracTickets for help on using tickets.