Show index for array's elements

litejs / json-lite

Fast non-blocking JSON viewer for Chrome and Firefox

195 stars 16 forks source link

Show index for array's elements #43

Open Karacala opened 5 years ago

Karacala commented 5 years ago

Show index for array's elements [i]

json_screenshot

zckrs commented 5 years ago

@lauriro any news about this PR

I think is a usefull feature and a option is available to activate the new feature

lauriro commented 5 years ago

First of all, thank you @Karacala for your contribution.

I have been thinking about this feature in silence and I am not completely satisfied yet.

Indices can be useful but not for arrays with 2-3 item. Always showing them is a lot of noise. I tried to show them on hover like date info, then start thinking, that json-pointer could be more useful.

Currently I am thinking about showing json-pointer on click.

zckrs commented 5 years ago

not for arrays with 2-3 item

Agree. Show indices on hover can resolved this issue.

json-pointer on click.

Would be add in another PR ?