Bug #75958

RTE - Show table borders in Firefox

Added by Guillaume Germain 10 months ago. Updated 10 months ago.

Status:Resolved Start date:2016-04-28
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:RTE (rtehtmlarea + ckeditor) Spent time: -
Target version:Candidate for patchlevel
TYPO3 Version:7 Complexity:
PHP Version:5.6 Is Regression:No
Tags: Sprint Focus:

Description

Hello,

In Firefox, the RTE button "Show table borders" does not work. There is a CSS bug.

You must write "!important" and not "! important" in "typo3_src-7.6.6/typo3/sysext/t3skin/rtehtmlarea/htmlarea-edited-content.css"

Associated revisions

Revision 32206e1e
Added by Tomita Militaru 10 months ago

[BUGFIX] Removes extra space from important

Removes extra space from between ! and important
override from rtehtmlarea CSS.

Resolves: #75958
Releases: master
Change-Id: If19ff8cc80c11fe650c12e914a07d5b6938a8ab8
Reviewed-on: https://review.typo3.org/48100
Reviewed-by: Wouter Wolters <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision e9f46233
Added by Tomita Militaru 10 months ago

[BUGFIX] Removes extra space from important

Removes extra space from between ! and important
override from rtehtmlarea CSS.

Resolves: #75958
Releases: master,7.6
Change-Id: If19ff8cc80c11fe650c12e914a07d5b6938a8ab8
Reviewed-on: https://review.typo3.org/48109
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>

History

#1 Updated by Riccardo De Contardi 10 months ago

  • Target version set to Candidate for patchlevel

#2 Updated by Riccardo De Contardi 10 months ago

  • Category set to RTE (rtehtmlarea + ckeditor)

#3 Updated by Riccardo De Contardi 10 months ago

In the file cited in the description there is actually a space between the exclamation mark and "important" (which is wrong) but as far as I can see, on Firefox (on Mac) it doesn't affect the "show table borders" visibility.

#4 Updated by Gerrit Code Review 10 months 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/48100

#5 Updated by Tomita Militaru 10 months ago

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

#6 Updated by Émilie Sagniez 10 months ago

Hello,

Is it possible to backport this patch in TYPO3 7.6 ?

#7 Updated by Gerrit Code Review 10 months ago

  • Status changed from Resolved to Under Review

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

#8 Updated by Gerrit Code Review 10 months ago

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

#9 Updated by Tomita Militaru 10 months ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF