[proposal] attach focus should accept an empty string

[bigopon]

dialog/src/attach-focus.ts

Line 18 in 0d35c2d

if (this.value && this.value !== 'false') {

The above line should be

if (this.value === '' || this.value && this.value !== 'false') {

I think we should make attach-focus boolean attribute for the following usage

<input attach-focus />

[bigopon]

This could be a breaking change, can we discuss it before official release @StrahilKazlachev @EisenbergEffect

[StrahilKazlachev]

@bigopon can you give a scenario where it will be a breaking change?

[bigopon]

When someone bind to a string, that may have value equals ''

  <input attach-focus.bind='message' />

Currently, It doesn't focus if message is ''

[Alexander-Taran]

@bigopon do something about it (-: