adopted-ember-addons / adopted-ember-addons/ember-light-table

Adding CSS classes to thead, tfoot and tbody

オープン
#492 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement help wanted ideas-wanted
主要言語
JavaScript
スター
311
フォーク
130
PR マージ指標
30日以内にマージされた PR はありません

説明

@fran-worley https://github.com/offirgolan/ember-light-table/issues/99#event-734291228:

> Sorry to bump an old issue but I believe this only partially resolves issues marrying up Bootstraps css styles with ember-light-table.
>
>We need to be able to apply styles directly to the 'thead' too ('.thead-default' is just one style example from BS v4). Could the api be harmonized to allow you to add classes directly to table elements (table, thead, tbody etc.).

As a user, I would expect that setting `class` on `{{t.head}}`, `{{t.body}}` and `{{t.foot}}` set these classes on the rendered ``, `` and `` elements.

I'm not sure, if we can use `class` as the attribute name here, since it's a magic `Component` keyword that automatically sets the `class` HTML attribute for the rendered component element.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

The issue names the contextual components t.head, t.body, and t.foot but no files or tests. Start by locating those component entry points and inspect how their class attributes are handled for the rendered thead, tbody, and tfoot elements. Done means classes supplied on each component appear on the corresponding table section without breaking existing usage.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
frontend
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。