Here is the list of things to fix before releasing v10.1.0:
[x] dynamic tables: give the focus to first input when adding a row
[x] fix npm run lint so it can handle lit linting errors
[x] it appears that lit-analyser, which we use in Codium, has some interesting hightlights. I did not managed to configure it, but it would be great.
[x] fix remaining lit linting errors
[x] dynamic table: add titles to "+" and "-" buttons
[x] dynamic table: add confirmation when deleting a line
[x] channel configuration: add a cancel button that reload data from backend
[x] channel emojis: add a cancel button
[x] in channel-detail, channel-configuration, ... there are some public attributes prefixed by _, and some private attributes without the prefix. Clean that.
[x] _asyncTaskRender: handle error (and pending?) case (this method exists in several elements)
[x] check that there is no missing localization for error messages (no hard coded labels)
[x] dynamic tables: column description on thead title attribute is not very accessible (does not work on mobile, is not easy to find if user don't know it exists)
[x] channel configuration: table too large?
[x] context in lit elements: seems a bit complicated. Should consider adding a "context provider" elements on each page, that provide various context, so that every other custom elements just have to consume them. Or just some singleton somewhere.
[x] .value=${this._channelConfiguration?.configuration.bot.enabled} => should be value="1". This error is present a few times. (not 100% sure of this)
[x] save button should be disabled during saving, to show the user something is going on if it takes some times (in channel-configuration and channel-emojis)
[x] dynamic table: always have at least an empty line? (i'm not sure of the impact)
[x] channel-configuration, channel-emojis, ...: source files are relatively big. Maybe we could separate the "template part" (content of the render method) in separate files. Like in ConverseJS.
[x] this._rowsById.filter(rowById => this.rows.includes(rowById.row)) <= seems buggy. Missing assignation of the result?
[x] classMap directive: documentation says it must be the only part, but there are some code that mix it with something else.
Here is the list of things to fix before releasing v10.1.0:
npm run lint
so it can handle lit linting errors.value=${this._channelConfiguration?.configuration.bot.enabled}
=> should bevalue="1"
. This error is present a few times. (not 100% sure of this)this._rowsById.filter(rowById => this.rows.includes(rowById.row))
<= seems buggy. Missing assignation of the result?