Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Thanks for the report! I used grep and found that these are normally now added in system_page_build().

So, sounds like a good novice project to fix up this code comment -- oh, I see you've already tagged the issue. :)

nesta_’s picture

Status: Active » Needs review
FileSize
616 bytes

I searched the string and is not used for what I chose to delete this comment, because the cast was obsolete information.

A greeting!

jhodgdon’s picture

Status: Needs review » Needs work

But... Those CSS files are normally added by system_page_build(), as noted in comment #1. Why not change the code comment to say that?

giammi’s picture

trying...

jhodgdon’s picture

Status: Needs review » Needs work

Thanks! This patch is better... but there are a couple of issues:
- The information in the original comment "except maintenance.css" was lost.
- There is a space at the end of the first line. If you plan to work on Drupal patches (and I hope you will continue!) see if you can either set up your editor to remove spaces at the end of lines, or at least highlight them.

nesta_’s picture

nesta_’s picture

Status: Needs work » Needs review

forget change status sorry.

nesta_’s picture

add "-except maintenance.css."

:)

giammi’s picture

Thanks for your feedback.

penyaskito’s picture

Status: Needs review » Reviewed & tested by the community

After feedback has been addressed I think we can RTBC this one.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x.

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