Project

General

Profile

Actions

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.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend User Interface
Target version:
Start date:
2014-11-01
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

refactor the reports module to use the correct kind of TWBS components instead of messages.


Files


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #62668: Optimize Reports ModuleClosedFrank Nägler2014-11-03

Actions
Actions #1

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

Actions #4

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

Actions #5

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

Actions #6

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

Actions #7

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

Actions #8

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

Actions #9

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

Actions #10

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

Actions #11

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

Actions #12

Updated by Frank Nägler over 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #13

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

Actions #14

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF