Task #65543

Looks error page frontend

Added by Job Rutgers almost 5 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2015-03-05
Due date:
% Done:

100%

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

Description

The looks of the error messages on the frontpage should be refreshed. With typo3-message message-error class.

error_page.png View (21.9 KB) Job Rutgers, 2015-03-05 14:41

error_before.png View (21.6 KB) Bjoern Jacob, 2015-11-19 16:34

error_after.png View (19.3 KB) Bjoern Jacob, 2015-11-19 16:34

blue-message.png View (62.9 KB) Harry Glatz, 2015-11-19 21:36

error_no_description.png View (5.02 KB) Bjoern Jacob, 2015-11-19 21:47

Associated revisions

Revision e1b1578a (diff)
Added by Markus Sommer almost 4 years ago

[TASK] Redesign TYPO3 error page

  • Error page in global TYPO3 CMS style
  • Error message in backend style
  • CSS refactor and cleanup

Resolves: #65543
Releases: master
Change-Id: Ia24ada1e11f578ed5bedd71bdbdb63843d7aa5b7
Reviewed-on: https://review.typo3.org/44243
Reviewed-by: Bjoern Jacob <>
Tested-by: Bjoern Jacob <>
Reviewed-by: Harry Glatz <>
Tested-by: Harry Glatz <>
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Frank Nägler <>
Tested-by: Frank Nägler <>

History

#1 Updated by Jigal van Hemert over 4 years ago

  • Category set to Backend User Interface

Although this is about the frontend error messages, the styling is directly related/retrieved from the backend styling; therefore the Category "Backend User Interface".

#2 Updated by Benni Mack over 4 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)

#3 Updated by Susanne Moog over 4 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#4 Updated by Benni Mack about 4 years ago

  • Target version changed from 7.5 to 7 LTS

#5 Updated by Gerrit Code Review about 4 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 https://review.typo3.org/44243

#6 Updated by Gerrit Code Review about 4 years ago

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

#7 Updated by Gerrit Code Review about 4 years ago

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

#8 Updated by Gerrit Code Review about 4 years ago

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

#9 Updated by Gerrit Code Review about 4 years ago

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

#10 Updated by Gerrit Code Review about 4 years ago

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

#11 Updated by Gerrit Code Review about 4 years ago

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

#12 Updated by Gerrit Code Review about 4 years ago

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

#13 Updated by Bjoern Jacob about 4 years ago

Check the attatched screenshots.

The last screenshot shows the vertically centered title. This is possible if .media-body gets a vertical-align: middle; and the p tag inside .callout-body is removed.

#14 Updated by Harry Glatz about 4 years ago

I added a message which exactly uses this code we have right now. so i think this is ok right now.

#15 Updated by Bjoern Jacob about 4 years ago

Your images are correct but are not the same. Because your messages have a description and that's why the title is pushed to the top. Within this ticket we've got no description and therefore only a title. Please see my attatched image (displays an alert which we also have in our error case).

#17 Updated by Harry Glatz about 4 years ago

you are right. but at least there is a marker for ###MESSAGE### - so template-wise there could be a message :)

As discussed in slack this vertical-align only makes sense if it is checked wether there is a message or not.

#18 Updated by Gerrit Code Review about 4 years ago

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

#19 Updated by Gerrit Code Review about 4 years ago

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

#20 Updated by Gerrit Code Review about 4 years ago

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

#21 Updated by Anonymous almost 4 years ago

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

#22 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF