Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Field lists have weird padding #149

Closed
FichteFoll opened this issue Sep 16, 2014 · 1 comment
Closed

Field lists have weird padding #149

FichteFoll opened this issue Sep 16, 2014 · 1 comment
Assignees
Labels
Bug A bug Needed: patch A pull request is required

Comments

@FichteFoll
Copy link

Source:

.. class: Test
    .. method: test()
        :param int point: Point in the view whose scope the selector should be
                          matched against.
        :param str selector: A scope selector.
        :returns bool: Whether the selector matches or not.

Current css:

.rst-content table.field-list td {
    border: medium none;
    padding-top: 5px;
}

2014-09-16_17 37 14

Without padding:

.rst-content table.field-list td {
    border: medium none;
}

2014-09-16_17 37 28

@f0k
Copy link

f0k commented Jun 2, 2015

+1! We've got the same problem in our documentation: http://lasagne.readthedocs.org/en/latest/modules/nonlinearities.html#lasagne.nonlinearities.sigmoid
The .rst-content table.field-list th has padding: 8px 16px, which gives a padding-top of 8 pixels.
The .rst-content table.field-list td sets padding-top: 5px, which gives a discrepancy of 3 pixels that really hurts. It should either set padding-top: 8px or just not set it at all, so it inherits the 8 pixels from .rst-content table.docutils td.

What can we do to fix this either in the theme, or for our documentation? (I don't feel comfortable to install the full ruby-based development environment, but I have a certain motivation to make our docs look good.)

PS: Thank you for the (otherwise extremely) good-looking theme!

/edit: The way to fix it for a single project is by overriding the CSS: See #117 (comment) and the following four comments. The CSS to add would be:

.rst-content table.field-list .field-body {
    padding-top: 8px;
}

This overrides the 5px specification because the selector is more specific. Would still appreciate a general solution.

@Blendify Blendify added Bug A bug Needed: patch A pull request is required labels Mar 10, 2017
@Blendify Blendify self-assigned this Mar 10, 2017
@agjohnson agjohnson added Working PR: work in progress Pull request is not ready for full review and removed Working labels Mar 14, 2017
@agjohnson agjohnson removed the PR: work in progress Pull request is not ready for full review label Mar 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Needed: patch A pull request is required
Projects
None yet
Development

No branches or pull requests

4 participants