I've seen the new design at localize.drupal.org and I just cant understand why are the comments displayd with bold fonts? (See for example here: http://localize.drupal.org/node/1778) It seems that coments are more important than the main article?!

It would be a better idea to show the new comments with bold fonts, but to use .comment .content {font-weight:bold;} as a main setting is a bad way IMHO!

CommentFileSizeAuthor
#6 no-bold-comments.patch436 bytesgábor hojtsy

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Wholeheartedly agree. This needs to be decided and implemented in the theme.

Bojhan’s picture

Bojhan commented

Already said this to lisarex and from my understanding that comments are bold isn't correct.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

https://infrastructure.drupal.org/drupal.org-style-guide/page_furniture.... does not list them being bold either. Why was it implemented bold then?

Bojhan’s picture

Bojhan commented

No reason, clearly a mistake - patch it?

lisarex’s picture

lisarex commented

Patch it, for sure.

I think the bolding was introduced when we were looking at the association theme, but it shouldn't be there. thanks!

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Needs work
Issue tags: +needs deployment on association.drupal.org
StatusFileSize
new no-bold-comments.patch436 bytes

Ok, committing this and deploying to l.d.o. To be deployed to association.d.o.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Needs work » Fixed

I'll consider it fixed when committed.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

@drumm: how do we track deployment then especially that association.d.o does not run with the trunk of the theme yet?

drumm’s picture

drumm
he/him
Location NY, US
commented

Updating Association to bluecheese trunk is a separate issue, that will fix quite a few issues.

Status: Fixed » Closed (fixed)
Issue tags: -font, -needs deployment on association.drupal.org, -font weight

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

Issue tags: +font, +needs deployment on association.drupal.org, +font weight

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.