Project

General

Profile

Actions

Defect #32829

closed

HTML entity is used in CSS string

Added by Alexander Sapozhnikov over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Ordering handles (green arrows ↕) at some administrative pages contains HTML entity   instead of inivisible character of non-breaking space:

Screenshot of page and inspector

Affected pages

  • /roles
  • /trackers
  • /issue_statuses
  • /enumerations
  • /custom_fields

Configuration

How to fix

Edit public/stylesheets/application.css:

.icon-only::after {
  content: " ";
}

replace   with

  • non-breaking space
    or
  • \a0

Files

redmine-410-nbsp.png (38.9 KB) redmine-410-nbsp.png Screenshot of page and inspector Alexander Sapozhnikov, 2020-01-17 01:09
application-css.patch (200 Bytes) application-css.patch Alexander Sapozhnikov, 2020-01-17 01:34
roles-no-problem@2x.png (48.6 KB) roles-no-problem@2x.png Go MAEDA, 2020-01-19 05:56
issue-32829-hidden.png (33.3 KB) issue-32829-hidden.png Hidden entities Alexander Sapozhnikov, 2020-01-19 21:56

Related issues

Related to Redmine - Patch #21805: Improve accessibility for icon-only linksClosedJean-Philippe Lang

Actions
Actions #1

Updated by Alexander Sapozhnikov over 4 years ago

--- application.css~    2019-12-20 17:19:46.000000000 +0500
+++ application.css    2020-01-17 05:28:57.819227647 +0500
@@ -1458,3 +1458,3 @@
 .icon-only::after {
-  content: " ";
+  content: "\a0";
 }
Actions #2

Updated by Go MAEDA about 4 years ago

I cannot reproduce the problem. Could you test again without third-party plugins and a theme?

Actions #3

Updated by Alexander Sapozhnikov about 4 years ago

I’ve checked with Alternate and Classic themes — there are no visible characters

Hidden entities

But anyway HTML entity should be replaced with allowed character or escape sequence.

https://developer.mozilla.org/en-US/docs/Web/CSS/string

Most characters can be represented literally. All characters can also be represented with their respective Unicode code points in hexadecimal, in which case they are preceded by a backslash (\). For example, \22 represents a double quote, \27 a single quote ('), and \A9 the copyright symbol (©).

Note: HTML entities (such as &nbsp; or &#8212;) cannot be used in a CSS <string>.

See also:

Actions #4

Updated by Go MAEDA about 4 years ago

  • Target version set to 4.1.1

Alexander Sapozhnikov wrote:

But anyway HTML entity should be replaced with allowed character or escape sequence.

https://developer.mozilla.org/en-US/docs/Web/CSS/string

Most characters can be represented literally. All characters can also be represented with their respective Unicode code points in hexadecimal, in which case they are preceded by a backslash (\). For example, \22 represents a double quote, \27 a single quote ('), and \A9 the copyright symbol (©).

Note: HTML entities (such as &nbsp; or &#8212;) cannot be used in a CSS <string>.

Thank you for giving me the information. I didn't know that.

Setting the target version to 4.1.1.

Actions #5

Updated by Go MAEDA about 4 years ago

  • Target version changed from 4.1.1 to 4.0.7
  • Affected version changed from 4.1.0 to 3.3.0

The change has been made in r15271 (Redmine 3.3.0). Updating the target version to 4.0.7.

Actions #6

Updated by Go MAEDA about 4 years ago

  • Related to Patch #21805: Improve accessibility for icon-only links added
Actions #7

Updated by Go MAEDA about 4 years ago

  • Subject changed from HTML entities instead of non-breaking space near ordering handles to HTML entity is used in CSS string
  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch. Thank you for posting the patch.

Actions

Also available in: Atom PDF