Calcite-components: Enhancement: showInput on calcite-tile-select refactoring

[driskull]

Summary

The showInput property has values of "left" | "right" | "none" but I think it should probably use start/end instead of left and right for RTL support.

It may also be a good time to do the following for it as well:

• reconsider property name
• Use interface for "start" | "end" | "none"

This would be a breaking change.

[macandcheese]

Are we ok with inputDisplay as the new name for this? cc @eriklharper

[driskull]

What about inputPosition? I'm ok with inputDisplay though.

[macandcheese]

-position does make sense (or -alignment) but since there's a "none" option that seems weird whereas with -display it's... slightly less weird? I don't feel super strongly 🤷‍♂️

[driskull]

I agree with those feelings. It's like it maybe should be split into two properties. One to deal with disabling the feature, and one to do with positioning. What do you think?

[macandcheese]

showInput (boolean) +
inputAlignment as new prop.. then we could Extract start / end from the Alignment interface instead having to create a unique one for start / end / none

[eriklharper]

showInput (boolean) +
inputAlignment as new prop.. then we could Extract start / end from the Alignment interface instead having to create a unique one for start / end / none

I like that idea, let's go with that!

[julio8a]

Great collaboration on this guys. What's the status of this? has the naming been updated?

[julio8a]

@macandcheese, is this is the issue you mentioned in our meeting that we can close?

Want to confirm.

[macandcheese]

Keep open until the linked pr is merged

[julio8a]

Linked PR was merged, can we close?