After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 709943 - yelp-xsl/xslt/common/html.xsl missing a fundamental docbook tag conversion
yelp-xsl/xslt/common/html.xsl missing a fundamental docbook tag conversion
Status: RESOLVED FIXED
Product: yelp-xsl
Classification: Core
Component: DocBook
3.6.x
Other Linux
: Normal normal
: ---
Assigned To: Yelp maintainers
Yelp maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-11 21:53 UTC by Doug Smythies
Modified: 2014-02-04 17:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot of keycap styling (2.15 KB, image/png)
2014-02-04 16:58 UTC, Shaun McCance
Details

Description Doug Smythies 2013-10-11 21:53:17 UTC
In Ubuntu-docs, both the DeskTop help and serverguide, the compiled HTML does not differentiate or emphasize some tag pairs as used to occur with the older theme.
An example of the issue is the fundamental docbook tag pair <keycap></keycap> doesn't get converted properly. However, I think the root issue is that these lines are missing from /usr/share/yelp-xsl/xslt/common/html.xsl:

span.key {
 font-weight: bold;
}

a suggested good place for them would be right around these lines:

span.key a {
 border-bottom: none;
}

Note: This issue has a workaround in that we have added those missing lines to our ubuntu.xsl files.

Ubuntu Launchpad bug report reference:
https://bugs.launchpad.net/serverguide/+bug/1173426
Comment 1 André Klapper 2013-10-11 23:31:46 UTC
Thanks for reporting this. Would you be able to provide a patch against git master of yelp-xsl?
Comment 2 Doug Smythies 2013-10-12 05:46:05 UTC
(In reply to comment #1)
> Thanks for reporting this. Would you be able to provide a patch against git
> master of yelp-xsl?

No.
Comment 3 Shaun McCance 2014-01-07 01:44:44 UTC
Problem is I switched the Mallard transforms to use <kbd> for <mal:key>, and changed the common CSS to reflect it:

https://git.gnome.org/browse/yelp-xsl/commit/?id=e4e160b2e4ab311e9c7a0a186adeb1b39da2de49

But I didn't change <db:keycap> over to use <kbd>, so it lost its styling.
Comment 4 Doug Smythies 2014-02-04 15:55:43 UTC
I tried yelp-xsl version 3.11.5. I didn't see the lines mentioned above added to html.xsl, nor was the desired effect present in the compiled html document after removing the workaround in ubuntu.xsl.
Comment 5 Shaun McCance 2014-02-04 16:57:58 UTC
The styling is done on the kdb element, not span.key. That's what the Mallard transforms switched to back in 2011 when I broke the DocBook styling. The DocBook transforms now output a kbd element, so they should pick up the styling. It's not "font-weight: bold". If that's how you want keys to be styled, then you need to maintain custom css for the HTML kdb element. Attaching a patch of what it should look like.
Comment 6 Shaun McCance 2014-02-04 16:58:20 UTC
Created attachment 268085 [details]
Screenshot of keycap styling
Comment 7 Doug Smythies 2014-02-04 17:37:35 UTC
Shaun thanks for your comments and the time you took to look into this.

For whatever reason, in the ubuntu serverguide the styling is almost imperceptible. I will leave the overrides I added to the both the Ubuntu Desktop and Serverguides ubuntu.xsl files in place. All I was trying to do was make things look similar to what they looked like a few years ago and make it obvious that it was a "key".