When displaying the page /admin/structure/taxonomy/manage/tags/overview:

Some tags are child of another tag. So Drupal add this:

<div class="js-indentation indentation">&nbsp;</div>

But, because it is displayed as "bock", the identation is not shown.

You need to add

.indentation {
   display: inline;
}

somewhere in the CSS.

Thanks
Jehon

CommentFileSizeAuthor
#12 bootstrap-2800569-12.patch1.17 KBpifagor
#11 bootstrap-2800569-11.patch1.86 KBpifagor
#9 bootstrap-2800569-9.patch688 bytespifagor
#6 bootstrap-2800569-6.patch642 bytespifagor
#5 bootstrap-2800569-5.patch640 bytespifagor
#3 bootstrap-2800569-3.patch270.05 KBpifagor

Comments

jehon created an issue. See original summary.

jehon’s picture

jehon commented
Issue summary: View changes
pifagor’s picture

pifagor commented
Status: Active » Needs review
StatusFileSize
new bootstrap-2800569-3.patch270.05 KB

patch

markhalliwell’s picture

markhalliwell
Primary language English
commented
Priority: Major » Normal
Status: Needs review » Needs work

Please do not provide a patch that modifies the compiled CSS. These are automatically generated as part of the maintainers grunt compile task from the less source files.

pifagor’s picture

pifagor commented
Status: Needs work » Needs review
StatusFileSize
new bootstrap-2800569-5.patch640 bytes

new patch

pifagor’s picture

pifagor commented
StatusFileSize
new bootstrap-2800569-6.patch642 bytes
markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs review » Needs work

Sorry. I also meant to mention that this should mimic the styles found in tabledrag.module.css.

So this should be added to both table-drag.less and _table-drag.scss, not form.less and _form.scss.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Title: Tags not properly aligned (js-identation is display-block) » Missing tabledrag indentations
pifagor’s picture

pifagor commented
Status: Needs work » Needs review
StatusFileSize
new bootstrap-2800569-9.patch688 bytes

new patch

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs review » Needs work

Per #7, this should use the existing styles from tabledrag.module.css, not the one from the OP:

.indentation {
  float: left; /* LTR */
  height: 1.7em;
  margin: -0.4em 0.2em -0.4em -0.4em; /* LTR */
  padding: 0.42em 0 0.42em 0.6em; /* LTR */
  width: 20px;
  [dir="rtl"] & {
    float: right;
    margin: -0.4em -0.4em -0.4em 0.2em;
    padding: 0.42em 0.6em 0.42em 0;
  }
}
pifagor’s picture

pifagor commented
Status: Needs work » Needs review
StatusFileSize
new bootstrap-2800569-11.patch1.86 KB
pifagor’s picture

pifagor commented
StatusFileSize
new bootstrap-2800569-12.patch1.17 KB

sorry

  • markcarver committed b251659 on 8.x-3.x authored by pifagor 
    Issue #2800569 by pifagor, jehon: Missing tabledrag indentations
  • markcarver committed cbe5320 on 8.x-3.x
    Automated commit: grunt compile (CSS)

Issue #2800569 by pifagor, jehon...
markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

pifagor’s picture

pifagor commented