Posted by amv101 on November 11, 2009 at 4:45pm
Jump to:
| Project: | Storm |
| Version: | 6.x-1.27 |
| Component: | Storm.module |
| Category: | feature request |
| Priority: | minor |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Is there a way to disable the icons and just leave the text links?
The module icons are causing weird cascading issues because of the design of my site.
thanks.
Comments
#1
There isn't currently a way, but it would be a simple method to implement a variable in the Storm admin settings and then check that within a few of the functions that output the icons.
So happy to commit that if it was default to display icons and done in a neat way.
#2
Cool... I think the icons are cool, dont get me wrong... But I am a designer and my site is deeply branded, so the icons dont make sense at all and conflict with my current branding. So I think having a choice would rock.
Even the add/remove/edit icons... maybe possible to have a switch for both. So use the alt text for the text link, instead of the icon/image?
#3
Alternatively, it occurs to me, you can replace the icons with your own - see the icons path setting.
But happy to have a switch.
#4
Well that is my issue. My site is very clean and icon-less. The icons for this purpose, take away from the brand. I am sure many other users are working within a brand too, so it might be a nice feature.
Also, from my production experience, when trying to get third party people, or new clients to use a system, a simple text link works much better than an icon, being you have to roll over the icon just to see what it is. As an accustomed user, icons can work, but when you have many new users, or many one time users, text links are the best option. Of course, unless the icon is a logo like PayPal.
#5
Just having a look at this.
Which images / icons do you mean?
Storm dashboard?
Links on the side of Storm nodes?
+ / - / edit links?
icons within tables?
all of the above?
#6
I think the +/-/edit icons are ok.... those are kind of universal.
But the rest would make since to have a text link / icon option, so it matches the rest of the site.
#7
Committed to Drupal 6, images on links and dashboard controlled by setting.
Todo:
- Copy to D7 branch.
- Apply to other images around Storm.
#8
Just tried this new feature and it work OK for me.
#9
Finished - committed to D7 and extended to other places where an image overrides text.
Actually works out very neatly in the code, it only changes about 6/8 lines.
@db102: thanks for the feedback!
#10
Glad to help out.
Now I notice that...
Icon for "Items Marked Unbillable" doesn't seem to turn off like the others. I notice it on View of Timetracking node.
Was this one left on for a reason?
#11
Reopening issue and will have a look.
#12
Done.
I think that covers everything - but let me know if there are any others that have been missed.
#13
Automatically closed -- issue fixed for 2 weeks with no activity.