vega / polestar

Lightweight Tableau-style interface for visual analysis, built on Vega-lite.

http://vega.github.io/polestar

Other

367 stars 47 forks source link

Remove caret from count #215

Closed domoritz closed 9 years ago

domoritz commented 9 years ago

Gives us more space

kanitw commented 9 years ago

I would just hide it. I would not remove otherwise you have inconsistent layout.

kanitw commented 9 years ago

pole

I also hide caret for text.

domoritz commented 9 years ago

Maybe we should move it to the right then. Then we can use the space and it doesn't look inconsistent.

kanitw commented 9 years ago

We moved it to the left due to inconsistency with "x"

domoritz commented 9 years ago

I remember. It should be fine if we have enough space between x and the caret, though.

kanitw commented 9 years ago

Also, I just made clicking "type" also toggle opening the popup too.

The popup allows you to change type and function. For proximity, I prefer it to be closer to what it changes.

kanitw commented 9 years ago

216 --> if we move things into field popup anyway, this point is moot.

kanitw commented 9 years ago

I think this can be closed. If you disagree, re-open.