Task #62614

Epic #62041: twbs Bootstrap backend, refactor EXT:t3skin and HTML5 output

Story #62606: Refactor FlashMessages to TWBS alternatives

Refactor Reports Module

Added by Frank Naegler over 5 years ago. Updated over 2 years ago.

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

100%

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.

New_TYPO3_site__TYPO3_CMS_7_0-dev_.png View - New Reports Module (277 KB) Frank Naegler, 2014-11-01 19:33

New_TYPO3_site__TYPO3_CMS_7_0-dev__und_Task__62614__Refactor_Reports_Module_-_Core_-_TYPO3_Forge.png View - Panels instead of messages (304 KB) Frank Naegler, 2014-11-01 19:35


Related issues

Related to TYPO3 Core - Task #62668: Optimize Reports Module Closed 2014-11-03

Associated revisions

Revision 8b944786 (diff)
Added by Frank Naegler over 5 years ago

[TASK] Message usage in reports module and extensionmanager

Refactoring the reports module a bit to no longer use status
message HTML for rendering since this is a misuse.

Similar change in extension manager configuration of salted
passwords extension.

Resolves: #62614
Releases: master
Change-Id: I0b3e4e1dfed72441bb01abdb4ad35b7dc058a606
Reviewed-on: http://review.typo3.org/33703
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

Revision 87f7d420 (diff)
Added by Felix Kopp over 5 years ago

[BUGFIX] Reports status listing table

Follow-up for Reports module test listing. Listing is a table.

Related: #62668
Related: #62614
Releases: master
Change-Id: Id51c1fe9e2961cb974511ed33666246d0c476ecf
Reviewed-on: http://review.typo3.org/33906
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Stefan Neufeind <>
Tested-by: Stefan Neufeind <>

History

#1 Updated by Gerrit Code Review over 5 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

#4 Updated by Gerrit Code Review over 5 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

#5 Updated by Gerrit Code Review over 5 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

#6 Updated by Gerrit Code Review over 5 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

#7 Updated by Gerrit Code Review over 5 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

#8 Updated by Gerrit Code Review over 5 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

#9 Updated by Gerrit Code Review over 5 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

#10 Updated by Gerrit Code Review over 5 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

#11 Updated by Gerrit Code Review over 5 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

#12 Updated by Frank Naegler over 5 years ago

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

#13 Updated by Felix Kopp over 5 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

#14 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF