Damien Cassou <damien@cassou.me> writes:

> David Bremner <david@tethera.net> writes:
>
>> David Bremner <david@tethera.net> writes: 
>> 
>>>> I did that in the patch even though I liked it with 
>>>> indentation better. 
>>> 
>>> I haven't had a chance to test the new version yet, but I think 
>>> I know what you mean from testing counsel-imenu. Is it worth 
>>> adding a customization variable so that the user can choose 
>>> indentation if they have a more sophisticated imenu front end? 
>> 
>> So this version is ok with both builtin and counsel imenu front 
>> ends. It's up to you. Do you want to leave the question of 
>> controllable indentation for a later commit or add it now? 
>
> if you are ok to merge that right now, that would be perfect for 
> me. Thanks. 

OK, done.

By the way, I notice a certain overlap of functionality between
counsel-imenu support and the way I use tree-view (Z) from show mode.
Given the small amount of code I think it's fine to support imenu as
well, but I thought I'd mention it.

d