Posted by Dane Powell on July 12, 2009 at 1:33am
| Project: | Pixture Reloaded |
| Version: | 6.x-3.2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Jeff Burnz |
| Status: | closed (fixed) |
Issue Summary
In Drupal error messages (div.error class), the text overlays the background "stop" image, making it unreadable. I think the text should be padded on the left like other Drupal messages are (like div.messages).
Comments
#1
Yes, I really must release another version with all these fiddly bits fixed up, thanks, I know about it but time is so precious right now - patches anyone?
#2
I tried to just add the necessary padding to the class definition in style.css, but it seems to be getting stripped out, I guess by the Color module. I don't know why or how to fix it.
#3
After changing the .css file, you have to go to the theme settings page and save the settings. You don't have to change anything, just save the settings again. That regenerates the color files and updates the the .css for your color scheme.
#4
Ah, okay. I was just flushing the caches and expecting the CSS to be regenerated. Thanks for the tip.
Here's a dead-simple patch for you.
#5
#6
Thanks, I'll commit to head in couple of days when I am home from holiday.
#7
It doesn't look like this has made it into head yet, has it? Just trying to close out some of my old issues here.
#8
I installed the theme properly but the customization for the different color sets via the color picker wont work. The theme just dosent set in. Only the default colour scheme works? What could be the problem and how can i correct this?
#9
Hi Talisuneo
Please don't change the issue title when posting a reply unless you are clarifying the original issue. Also, don't mark an issue as critical unless it is seriously crippling the functionality of a module. In this case, there's no way that my patch could have broken anything unless you applied it wrong.
I would suggest flushing all Drupal caches, go to the Pixture reloaded settings, save them (even if you don't make any changes - this recopies the necessary theme files as mentioned in #3), reflush Drupal caches, and then clear your browser cache and reload the page.
Edit: it sounds like your issue is wholly unrelated to this one- if that's the case, you should create a new issue using the link at the top of the project issue queue.
#10
@talisuneo - HELLO? Start a separate issue thanks very much, don't hijack this one for your own purpose!
#11
Forgot to set the assignment...
#12
Hello dane
Thank you so much. Its working fine now. I thought it had something to do with my database.
I needed to flush drupal cache and my browser cache too and its loading fine now.
And i apologize for posting my issue here. I am a complete newbie and this is my first post. Sorry for the critical issue mark too. I am deeply sorry guys. sorry Jeff
Thank you Dane again
cheers
#13
#14
Anything stopping this from being committed?
#15
#4 - Thanks for patch, it hepls me!
#16
I have marked #683350: Background image is used when not necessary as duplicate of this report.
#17
OK, I am hearing you guys! I'll get onto these issues, soon, right now I'm pretty slammed with D7 but I understand that I really really need to roll a new version of PR as there quite a number of issues that need to be cleaned up.
#18
subscribing
#19
subscribing
#20
this will be fixed in 6.x-4.0
#21
Automatically closed -- issue fixed for 2 weeks with no activity.