Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The capitalization is not very consistent and does not follow Drupal's standard.
Need to standardize it.
Comment | File | Size | Author |
---|---|---|---|
#6 | performance_install.patch | 6.88 KB | jcmarco |
#6 | performance_module.patch | 6.54 KB | jcmarco |
#3 | performance_module_coder.patch | 7.48 KB | jcmarco |
#3 | performance_install_coder.patch | 4.36 KB | jcmarco |
Comments
Comment #1
kbahey CreditAttribution: kbahey commentedMoving to devel queue, since the module is now part of that.
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedI welcome patches for coding standards.
Comment #3
jcmarco CreditAttribution: jcmarco commentedUsing Coder
Comment #4
moshe weitzman CreditAttribution: moshe weitzman commented@kbahey - look OK to you?
Comment #5
kbahey CreditAttribution: kbahey commentedjcmarco, thank you. I committed it for 6.x.
Can you be so kind as to download 7.x-1.x-dev tomorrow (because I just committed a change that will be packaged in about 9 hours from now) and do the same for it and submit a patch?
Comment #6
jcmarco CreditAttribution: jcmarco commentedChecked with last coder 7.x-dev
performance.install:
practically the same with just a correction in @file
performance.module:
added a recommendation for new drupal_module_exist.
I haven't changed the time() recommendation as I guess that it is necessary accuracy with timings.
Comment #7
jcmarco CreditAttribution: jcmarco commentedComment #8
kbahey CreditAttribution: kbahey commentedthanks you!
Comment #10
mycat CreditAttribution: mycat commentedfirst, sorry for my english.
in the devel-module, in devel_themer_ie_fix.css is an little error.
original Code (:
This text has also effect: "Themer Info" is not formatted as "bottom" ...
And: with absolute position the "Themer Info" is scrolling with the Layout.
must be:
fixed: "Themer Info" is too in IE always bottom!
With position fixed the "Themer Info" do not scroll with the Layout!
Thanks an greetings
Fredi
P.S. i hope, this is the right place for this message.
Comment #11
jcmarco CreditAttribution: jcmarco commentedProbably, the best would be to open a new issue
Comment #12
moshe weitzman CreditAttribution: moshe weitzman commented