<p></p>
<p><b>@elextr</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/2930#discussion_r736056727">src/highlightingmappings.h</a>:</p>
<pre style='color:#555'>> @@ -388,7 +388,12 @@ static const HLStyle highlighting_styles_CSS[] =
        { SCE_CSS_EXTENDED_IDENTIFIER,          "extended_identifier",                FALSE },
        { SCE_CSS_EXTENDED_PSEUDOCLASS,         "extended_pseudoclass",               FALSE },
        { SCE_CSS_EXTENDED_PSEUDOELEMENT,       "extended_pseudoelement",     FALSE },
-       { SCE_CSS_MEDIA,                                        "media",                                      FALSE }
+       { SCE_CSS_GROUP_RULE,                           "group_rule",                         FALSE },
+       /* In Geany 1.38 and earlier shipped filetypes.css with "media" identifier,
+        * Scintilla/Lexilla 5.1.2 has renamed this style and extended its meaning.
+        * We still recognize media for compatibility.
+        */
+       { SCE_CSS_GROUP_RULE,                           "media",                                      FALSE }
</pre>
<p>I had some time to check again (after reading W3C tutorial on <a class="user-mention" data-hovercard-type="organization" data-hovercard-url="/orgs/media/hovercard" href="https://github.com/media">@media</a> etc) and I agree with <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/eht16/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/eht16">@eht16</a>, using %Y in the status bar I can see that <code>media</code> and <code>document</code> in <code>@media</code> and <code>@document</code> is styled 22 but the applied named style is <code>default</code> (after editing <code>filetypes.common</code> to make<code>default</code> and <code>parameter</code> named styles blindly obvious :), so as usual <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/eht16/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/eht16">@eht16</a> is right ;-P.   But if I remove the line above, the applied named style is <code>parameter</code>.</p>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> it was a good idea, but it doesn't work.  I don't think its necessary to retain backward compatibility with syntactic entity names, previously <code>media</code> and now <code>group-rule</code> is mapped to the <code>parameter</code> named style which is what the colour schemes should be changing, so they will still work.</p>
<p>It would only be when someone who changed the named style that <code>media</code> mapped to in their <code>filetypes.css</code> that would not "just work" but they would probably be smart enough to change it again when the new Geany comes out especially if its highlighted in the next release NEWS.  Maybe add it to NEWS now so its not forgotten.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2930#discussion_r736056727">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ6MZKY6KLNIYPUCAKDUIXY3JANCNFSM5FYCAC7Q">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ3EB7K3ZKSL2XDB45LUIXY3JA5CNFSM5FYCAC72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOF4BRSLI.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2930#discussion_r736056727",
"url": "https://github.com/geany/geany/pull/2930#discussion_r736056727",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>