Project

General

Profile

Actions
Copy link

Feature #75466

closed

Add a class to the parent wrapper of fluid_styled_content

Added by Elmar Hinz about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
-- undefined --
Assignee:
-
Category:
Fluid Styled Content
Target version:
-
Start date:
2016-04-08
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:
On Location Sprint

Description

The parent wrapper of fluid styled content elements have an id but no class:

<div id="xxx">
   <h1>....</h1>

</div>

As they build a logical class it would make sense to have a common html class to address them in common:

<div id="xxx" class="ce">
   <h1>....</h1>

</div>
Actions
Copy link
 #1

Updated by Riccardo De Contardi about 8 years ago

I will repeat my opinion on this topic:
if this class means something in the CSS shipped by default with Fluid Styled Content, then it would be correct to add it by default, otherwise there is no "default" (why "ce" and not "fsc-default"? or "t3-element" or even "clearfix", why not? et cetera ... I hope I made myself clear.

If this class is necessary for the specific needs of the integrator, he/she can (very easily) add it overwriting the Templates/Layouts.

Actions
Copy link
 #2

Updated by Riccardo De Contardi almost 8 years ago

  • Category set to Fluid Styled Content
Actions
Copy link
 #3

Updated by Anja Leichsenring almost 8 years ago

  • Sprint Focus set to On Location Sprint
Actions
Copy link
 #4

Updated by Tymoteusz Motylewski over 7 years ago

  • Status changed from New to Needs Feedback

Hi Elmar,
Thanks for reporting the issue.
Can you elaborate more on the use case, where you need all elements to have a common class?
Thanks

Actions
Copy link
 #5

Updated by Gerrit Code Review about 7 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 66 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #6

Updated by Gerrit Code Review about 7 years ago

Patch set 67 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #7

Updated by Gerrit Code Review about 7 years ago

Patch set 68 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #8

Updated by Gerrit Code Review about 7 years ago

Patch set 69 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #9

Updated by Gerrit Code Review about 7 years ago

Patch set 70 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #10

Updated by Gerrit Code Review about 7 years ago

Patch set 71 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #11

Updated by Gerrit Code Review about 7 years ago

Patch set 72 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #12

Updated by Gerrit Code Review about 7 years ago

Patch set 73 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #13

Updated by Gerrit Code Review about 7 years ago

Patch set 74 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #14

Updated by Gerrit Code Review about 7 years ago

Patch set 75 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #15

Updated by Gerrit Code Review about 7 years ago

Patch set 76 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #16

Updated by Gerrit Code Review about 7 years ago

Patch set 77 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #17

Updated by Gerrit Code Review about 7 years ago

Patch set 78 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #18

Updated by Gerrit Code Review about 7 years ago

Patch set 79 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #19

Updated by Gerrit Code Review about 7 years ago

Patch set 80 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #20

Updated by Gerrit Code Review about 7 years ago

Patch set 81 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #21

Updated by Gerrit Code Review about 7 years ago

Patch set 82 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #22

Updated by Gerrit Code Review about 7 years ago

Patch set 85 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #23

Updated by Gerrit Code Review about 7 years ago

Patch set 86 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51065

Actions
Copy link
 #24

Updated by Anonymous about 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #25

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF