Size limit on topic status icon

Daemonite / discourse-material-theme

Material Design for Discourse

MIT License

21 stars 28 forks source link

Closed amotl closed 5 years ago

amotl commented 5 years ago

Hi @sesemaya,

we just wanted to drop another minor issue we found while implementing your theme.

The topic status icon is slightly off.

/* Fix DMT#7: Remove size limit from topic status icon */
.topic-statuses i.d-icon {
    height: unset !important;
    width: unset !important;
}

Thank you in advance for having a look.

With kind regards, Andreas.

sesemaya commented 5 years ago

I can't reproduce this problem here. Does this happen on a specific page with certain settings turned off/on?

amotl commented 5 years ago

Thanks. After checking this again, the displacement problem we saw before seems to have disappeared, both on vanilla and with our amendments.

However, we found it pleasant to add some spacing here.

/* Divert DMT#7: Adjust spacing between topic status icon and topic title */
.topic-statuses i.d-icon {
    margin-right: 0.25em;
}

amotl commented 5 years ago

Thanks, @sesemaya!