benjaminkott / benjaminkott/bootstrap_package
Unneeded DIV tags for content element shortcut / "Insert Records"
- Dominant language
- PHP
- Stars
- 355
- Forks
- 212
- Avg merge
- 1h 4m
- Merged PRs (30d)
- 31
Description
# Bug Report
## Prerequisites
* [x] Can you reproduce the problem on TYPO3 v11.5 LTS
* [x] Can you reproduce the problem on TYPO3 v12.4 LTS
* [x] Did you [perform a cursory search](https://github.com/issues?q=is%3Aissue+user%3Abenjaminkott+repo%3Abenjaminkott%2Fbootstrap_package)
to see if your bug or enhancement is already reported?
## Description
When using content element "Insert Records", first the "Insert Record" content element is wrapped in many div tags, then the referenced content element, too:
```
```
This not only bloats the source code, it's also an unexpected result (when styling content elements with CSS, this HTML structure is unexpected in my opinion), and this leads to double paddings for this content element (this is what I meant with "unexpected", you didn't expect this structure when writing styles for ".frame" class ;) ):
The red "1" is first padding, "2" second padding.
I will provide a fix for this.
## Steps to Reproduce
1. Use content element "Insert Records"
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reproducing the TYPO3 “Insert Records” content element and inspecting its generated HTML. Trace the rendering path for the nested content element; done means the redundant wrapper divs and resulting double padding are removed without breaking the referenced content output.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- bootstrap, php
- Domain
- frontend, web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100