Openlibrary: read-options panel is misaligned

Created on 2 Oct 2018  路  12Comments  路  Source: internetarchive/openlibrary

panel is misaligned

dropper

CSS

All 12 comments

Can you post a link of where you are seeing this so I can replicate easily?

It surely is on dev instance not really sure about prod

What page on dev instance are you looking at ?

From what I can see, the issue here is this rule:
.Tools .btn-notice { margin: 35px; }

Your patch only fixes this because #read-options is a more specific selector than .Tools .btn-notice (ids trump classes)

I think instead of overriding in this fashion it would be better to remove/update the .Tools .btn-notice rule.

Here is a screenshot with margin: auto. It competes with .Tools .btn-notice and takes precedence as I wrote in [pull]

details

Leaving out margin-left does not help:

without_margin-left

Instead adding margin: auto:

with_margin

Is that clear?

What page on dev instance are you looking at ?

From what I can see, the issue here is this rule:
.Tools .btn-notice { margin: 35px; }

Your patch only fixes this because #read-options is a more specific selector than .Tools .btn-notice (ids trump classes)

I think instead of overriding in this fashion it would be better to remove/update the .Tools .btn-notice rule.

This is wrong as .Tools .btn-notice does not apply here.

Is the margin of 35px on .Tools .btn-notice needed anywhere? If so, where?

I can only see class="btn-notice" in macros/databarWork.html and it is referenced in:

  • css/components/editions.less
  • css/components/Tools.less

Macro databarWork.html is used in templates:

  • openlibrary/templates/type/edition/view.html
  • openlibrary/templates/type/work/view.html
  • openlibrary/templates/type/edition-history/view.html

but nowhere with that order namely .Tools > .btn-notice so I'd say that rule is not used.

Thanks for the detective work!

I can't replicate this any more... did this get fixed?

I can't reproduce anymore. Voting to close

Was this page helpful?
0 / 5 - 0 ratings