{{cssinfo}} doesn't use <code> for media values

RESOLVED INVALID

Status

Developer Documentation
Macros/Templates
P5
normal
RESOLVED INVALID
3 years ago
3 years ago

People

(Reporter: teoli, Unassigned)

Tracking

Details

(Reporter)

Description

3 years ago
:: Developer Documentation Request

      Request Type: Correction
     Gecko Version: unspecified
 Technical Contact: 

:: Details

Media values: visual

Visual is in normal text and should be in code (like before)

E.g. https://developer.mozilla.org/en-US/docs/Web/CSS/left
(Reporter)

Updated

3 years ago
Blocks: 1198963
Note that this change was done intentionally. The media values are not related to code, they are just additional information like the value for 'Applies to'.
So I would rather close this issue as INVALID.

Sebastian
(Reporter)

Comment 2

3 years ago
I'm not sure:
* media values can be used in media queries (they are a subset of these). (Not 100% sure here)
* if they are textual, we need to translate them.

I don't want to over-engineer here as these things are in the progress of being phased out of CSS anyway.
(Reporter)

Comment 3

3 years ago
(My last sentence meant: make the call, Sebastian :-) )
(In reply to Jean-Yves Perrier [:teoli] from comment #2)
> I'm not sure:
> * media values can be used in media queries (they are a subset of these).
> (Not 100% sure here)

Media values may not match media types. E.g. most of the properties have a media value of 'visual', which is not a media type.

> * if they are textual, we need to translate them.

Already done in https://developer.mozilla.org/en-US/docs/Template:L10n:CSS.

(In reply to Jean-Yves Perrier [:teoli] from comment #3)
> (My last sentence meant: make the call, Sebastian :-) )

As we discussed earlier, we need a page describing the media types. See bug 1133414. Once we have that page, we should link to it from 'cssinfo', for which I now created bug 1214213.

Due to the above statements I close this report now.

Sebastian
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.