Task #62614
closed
Epic #62041: twbs Bootstrap backend, refactor EXT:t3skin and HTML5 output
Story #62606: Refactor FlashMessages to TWBS alternatives
Refactor Reports Module
Added by Frank Nägler over 9 years ago. Updated over 6 years ago.
100%
Description
refactor the reports module to use the correct kind of TWBS components instead of messages.
Files
| New_TYPO3_site__TYPO3_CMS_7_0-dev_.png (277 KB) New_TYPO3_site__TYPO3_CMS_7_0-dev_.png | New Reports Module | Frank Nägler, 2014-11-01 19:33 | |
| New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png (304 KB) New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png | Panels instead of messages | Frank Nägler, 2014-11-01 19:35 |
Updated by Gerrit Code Review over 9 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Frank Nägler over 9 years ago
Updated by Frank Nägler over 9 years ago
- File New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png added
- File New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png added
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Gerrit Code Review over 9 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/33703
Updated by Frank Nägler over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 8b9447860c76a5f82a3ae9194e0709c1c164fd3c.
Updated by Felix Kopp over 9 years ago
Why did you not create a single table for headline? Currently the output is not improved from my opinion. The view is to long, sections can not be seen/scanned quickly. The view is worse in my opinion since users now have to read the headline, then the line below and after that read to the right for every line! Top, down, right, top, down, right.
The table before had a good structure: read the headlines top to bottom or read the status result top to bottom.
Also the semantic structure is not correct because message title is not the table header. Table header should describe the type of content of the cell - because of that the table headers are not on the status color bg.
I am not happy with this result and propose to create one table per section/headline.
+ Add table header for the type (left: Test, right: Status).
+ Create a table row per test with two cells. Long description text should be in Status or Test col
Updated by Riccardo De Contardi over 6 years ago
- Status changed from Resolved to Closed