fsubal
fsubal
Adding `touchstart` events via Snap.svg leads to the following warning (at Chrome 70)  This is because `addEvent` in mouse.js always passes `false` as the third argument...
### Problem When you start deploying a branch and someone push to that branch immediately after that, you will see the deploy fails ( when `shallowClone: true` ). ``` Checking...
## やったこと - independent versioning をやめる - これからはすべての package の最新版は全く同じバージョンになる ## 動作確認環境 ## チェックリスト 不要なチェック項目は消して構いません - [ ] 破壊的変更がある場合には、対象のパッケージのメジャーバージョンが上がっていることを確認した - [ ] 追加したコンポーネントが index.ts から再 export されている
## やったこと ref: https://github.com/pixiv/charcoal/pull/240 Yarn の PnP モードで得られる「本来追加すべき dependencies のチェックが厳密になる」というメリットは pnpm でも得られる。yarn と違ってエディタ向けの特殊対応とかも必要がない。 そして、pnpm の方が OSS で見かけるのでこっちにしたらどうなるか検証したい（ next.js のリポジトリも pnpm を使っている ） ## 動作確認環境 CI が通ること、手元でエディタが普通に動くこと、Storybook が起動すること ## チェックリスト 不要なチェック項目は消して構いません...
### 必要だと思う機能の内容を書いてください 各パッケージの依存のインストール忘れを検知できるようにするツールを使いたい。 選択肢としては以下の通り - Yarn の PnP モードに移行する - https://github.com/pixiv/charcoal/pull/240 - Yarn をやめて pnpm に移行する - https://github.com/pixiv/charcoal/pull/250 - パッケージマネージャはそのままで、eslint-plugin-import などでおかしな import を検知する - https://zenn.dev/yusukehirao/articles/no-extraneous-dependencies ### この機能が必要と考えられる理由を書いてください https://github.com/pixiv/charcoal/pull/226 現状だとモノレポ内のすべてのパッケージが同じ node_modules...
### 必要だと思う機能の内容を書いてください tailwind-config と styled で CSS Variables の命名を揃えたい ### この機能が必要と考えられる理由を書いてください tailwind-config と styled はほぼ同じ CSS Variables を提供しているのに、命名が違う。 したがって、`@charcoal-ui/tailwind-config` と `@charcoal-ui/react` を両方使っているリポジトリでは、無駄に多くの CSS Variables が定義されてしまう。 ### この機能があった場合にどういう動きをするかの例を書いてください `@charcoal-ui/utils` に変数名を組み立てる良い感じの関数があって、両方が同じものを使うようになる。 一応...
### できる限り詳細に、不具合の内容と再現方法を報告してください Next.js で charcoal-ui/react の `` コンポーネントを利用しているページで奇妙な挙動が起きる。 別のページから SPA 遷移してきたときには Button が正しい見た目で表示されるが、ページをリロードするとスタイルが当たらなくなってしまう。 インスペクタで見ると `Button` の color や background-color が `clickableCss` に上書きされて inherit になってしまっていることがわかる。スタイルの優先順位が不定になっていそう？  https://github.com/pixiv/charcoal/blob/d2e8ec97fd936ddbecc29f5f84b78c2265b3647c/packages/react/src/components/Clickable/index.tsx#L92 --- これに関連していそう。 [styled-componentsの仕組みについての覚え書き |...
For gradual adoption, it may be helpful to support something like `// irontail-ignore` or `// irontail-ignore-next-line` comment.
cf: https://github.com/tailwindlabs/tailwindcss-intellisense/blob/6659228975393fb583268b50c2274d082a159072/tests/extractClassNames.test.js Doing just `Object.keys(result.classNames)` may be wrong ( It might be nested ). https://github.com/fsubal/irontail/blob/master/src/TailwindClient.ts#L56
Resolves: https://github.com/fetch/node-sass-asset-functions/issues/16 ------ I made this library to be sass-implementation agnostic, that means, add support for `dart-sass`. With these changes, node-sass-asset-functions now accepts `implementation` option ( which excepts `require('node-sass')` or...