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

Magnity’s picture

Magnity commented

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.

amv101’s picture

amv101 commented

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?

Magnity’s picture

Magnity commented

Alternatively, it occurs to me, you can replace the icons with your own - see the icons path setting.

But happy to have a switch.

amv101’s picture

amv101 commented

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.

Magnity’s picture

Magnity commented

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?

amv101’s picture

amv101 commented

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.

Magnity’s picture

Magnity commented

Committed to Drupal 6, images on links and dashboard controlled by setting.

Todo:
- Copy to D7 branch.
- Apply to other images around Storm.

dbt102’s picture

dbt102 commented

Just tried this new feature and it work OK for me.

Magnity’s picture

Magnity commented
Status: Active » Fixed

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!

dbt102’s picture

dbt102 commented
Version: 6.x-1.9 » 6.x-1.27
Priority: Normal » Minor

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?

Magnity’s picture

Magnity commented
Status: Fixed » Active

Reopening issue and will have a look.

Magnity’s picture

Magnity commented
Status: Active » Fixed

Done.

I think that covers everything - but let me know if there are any others that have been missed.

Status: Fixed » Closed (fixed)

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