Closed (fixed)
Project:
iTweak Upload
Version:
6.x-2.2
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
28 Aug 2009 at 04:35 UTC
Updated:
6 Oct 2009 at 15:10 UTC
See screenshots, one is attachment uploading in node, the other one - in comments and this one doesn't look right. Is it supposed to be that way?
iTweak Upload 6.x-2.2
Comment Upload 6.x-1.x-dev from 2009-Jul-15
| Comment | File | Size | Author |
|---|---|---|---|
| appearence-comments.jpg | 79.15 KB | batbug2 | |
| appearence-node.jpg | 48.09 KB | batbug2 |
Comments
Comment #1
iva2k commentedNo, the comments do not look right.
You may be experiencing CSS limit issue (IE and FF browsers have small limit of CSS files that they would load for any given page, like 30 or 31). Any "unlucky" CSS file that ends up ignored would cause such rendering issues. Try enabling "Optimize CSS files" in admin/settings/performance, then flushing all caches and checking if problem remains. If it goes away, your site indeed gets more than the max limit of CSS files.
This is a well known drupal problem, where each module suppose to provide a separate CSS file if it uses custom styling, and many modules do that.
Comment #2
batbug2 commented"Optimize CSS files" is always enabled
Comment #3
iva2k commentedTry flushing all caches and report if there is any change. If not, can you give a url to your website (if sensitive, can do a private message). I can check what is going wrong. Also, what browser / version are you using?
Comment #4
batbug2 commentedFlushing the caches does not help.
Site URL is http://liveangarsk.ru
This happens in all browsers.
And by looking at the code this seems to be not just CSS issue: the differences between those two upload forms is their IDs: table#upload-attachments and table#comment-upload-attachments. I added neccessary lines to the css, and now the tables look kinda the same, but still in comment uploading the attached files are not in the table but below it.
My ICQ is 993111, and I guess we could both speak Russian (i'm always online, just in the invisible mode).
Comment #5
batbug2 commentedComment #6
batbug2 commentedOk, i have fixed it, weird way though...
In comment_upload.module changed theme_comment_upload_form_current function to this
This made it all work correctly but regular, non-image attachment list was printed twice in a comment.
Had to change a bit in itweak_upload.module:
In function itweak_upload_comment(&$comment, $op) i commented out
Now it all works like advertised! (Of course i had to add #comment-upload-attachments id to the itweak_upload.css file where needed.
Comment #7
iva2k commentedI guess website @ the URL is not showing the problem anymore. Also, I did not find which nodes have comments with attachments by browsing - it would be more helpful to have complete URL.
Your fix suggests that comment_upload.module theme function theme_comment_upload_form_current() is called instead of itweak_upload.module itweak_upload_comment_upload_form_current(), which should not happen if itweak_upload is enabled. That can explain why it was not displaying attachments properly in comments. My first guess (flush all caches) was about that. Actually you should rebuild theme registry, which flushing all caches is supposed to be doing.
It could also be due to module weights in {system} table, perhaps if comment_upload.module weight was changed from its default. If it is the case, a better fix would be to rearrange weights. Supporting evidence for that being the case is in #4 table#comment-upload-attachments. itweak_upload does not create table#comment-upload-attachments. It creates table#upload-attachments for both node and comment_upload files, so they are styled by the same CSS.
Please try changing module weights (itweak_upload weight should be > comment_upload weight).
Comment #8
batbug2 commentedLatest updates to comment_upload module set its weight to 2. Itweak's weight is 0.
I changed Itweak's weight from 0 to 50, rebuilt theme registry and reverted all my hacks. Now it all works as expected :) Thank you very much.
Though I've encountered another small glitch: in this node - http://liveangarsk.ru/blog/fantazmer/20090907/mms-subject-vo - attached there photo doesn't show up as a thumbnail, because this node was created with mailhandler. It works fine in a regular nodes (like http://liveangarsk.ru/blog/batbug/20090904/zubik )/ I find it very strange because Itweak_upload invokes hook_nodeapi on NODE VIEW and this photo must be processed. Any clue?
Comment #9
iva2k commented@batbug:
Thanks for the test & update. I will follow up with appropriate iTweak Upload changes.
Note to self:
Need to make iTweak Upload weight > comment upload weight.
Re: "Though I've encountered another small glitch: in this node..."
Please open a separate issue for that. It is better to track separate issues in separate threads.
Comment #10
iva2k commentedFixed in 6.x-2.x-dev.