Author: Raphael Isemann Date: 2021-01-20T09:05:28+01:00 New Revision: 7fce3b240b6b313b1becf19ddf3f2a904c34ced2
URL: https://github.com/llvm/llvm-project/commit/7fce3b240b6b313b1becf19ddf3f2a904c34ced2 DIFF: https://github.com/llvm/llvm-project/commit/7fce3b240b6b313b1becf19ddf3f2a904c34ced2.diff LOG: [lldb][docs] Remove -webkit-hyphens in table cells so that table widths are correct on Safari The tables in the new LLDB documentation currently are less wide than their contents. The reason for that seems to be the `-webkit-hyphens: auto` property that sphinx is setting for all `p` tags. The `p` tags in the generated Python documentation seem to trigger some Safari layout issue, so Safari is calculating the cell width to be smaller than it should be (which ends up looking like this {F15104344} ). This patch just sets that property back to the browser default `manual`. Not sure if that's the proper workaround, but I clicked around on the website with the changed CSS and nothing looked funny (which is I believe how webdev unit testing works). Reviewed By: JDevlieghere Differential Revision: https://reviews.llvm.org/D94991 Added: Modified: lldb/docs/_static/lldb.css Removed: ################################################################################ diff --git a/lldb/docs/_static/lldb.css b/lldb/docs/_static/lldb.css index 53b725940831..6df20b7dd6e4 100644 --- a/lldb/docs/_static/lldb.css +++ b/lldb/docs/_static/lldb.css @@ -44,6 +44,13 @@ table.mapping td.content { padding-bottom: 15px; } +/* Workaround for a Safari bug that would otherwise make table cells less wide +than the containing text. This just sets it back to the default browser +property.*/ +td p { + -webkit-hyphens: manual !important; +} + div.sphinxsidebar .caption { font-family: Helvetica, Verdana, sans-serif; font-size: 10pt; _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits