Visually, I think this is very appealing.  Nice work!

Two quick questions:

Is there something keeping us from removing this completely?

~~~~~
allura/lib/macro.py

272:     display_mode=None, grid_view_tools='',  # old & unused now
287:     display_mode=None, grid_view_tools='',  # old & unused now

~~~~~


2) Do you think this should be in the ini? Or we could just remove it 
completely.


~~~~~
allura /templates/jinja_master/master.html:36
{#  {% do g.register_css('/nf/tool_icon_css?' + g.build_key, compress=False) %} 
  If you need tool icons, use this #}
~~~~~



---

** [tickets:#8004] Remove tool icons from project nav**

**Status:** review
**Milestone:** unreleased
**Labels:** ux sf-current sf-4 
**Created:** Fri Oct 16, 2015 08:42 PM UTC by Dave Brondsema
**Last Updated:** Thu Oct 22, 2015 08:00 PM UTC
**Owner:** Dave Brondsema


Per discussion at thread 
http://mail-archives.apache.org/mod_mbox/allura-dev/201510.mbox/%3CCAOH-DerrdKor_BdfecYXoF0sfiZ%3DK893%3DzmQiet%2B%2Bc_-8vMUSw%40mail.gmail.com%3E


---

Sent from forge-allura.apache.org because [email protected] is subscribed 
to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.

Reply via email to