cssinjs/jss

Previous styles not being removed when returning 'null' values using dynamic rule sets.

Open

#1,421 建立於 2020年11月7日

在 GitHub 查看
 (2 留言) (0 反應) (0 負責人)JavaScript (7,000 star) (434 fork)batch import
bughelp wanted

描述

NOTE: Similar to https://github.com/cssinjs/jss/issues/1026, but with dynamic rule sets instead of dynamic values.

Just to clarify nomenclature... dynamic rule set Vs. dynamic value:

{
  // DYNAMIC RULE SET
  a: ({color}) => ({
    display: 'inline',
    color: color
  }) ,
  b: {
    display: 'inline',
    // DYNAMIC VALUE
    color: ({ color }) => color
  }
}

Expected behavior: Previous applied values to be removed when respective value is set to null in a dynamic rule set.

Describe the bug: Previous values are not removed causing a number of trouble.

Codesandbox link: https://codesandbox.io/s/fervent-dawn-uomkr

Versions:

  • JSS: 10.4.0
  • Browser: All
  • OS: macOS 10.15.7

貢獻者指南

技術棧
javascriptcss
領域
frontend
議題類型
bug
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 hours
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
Familiarity with JSS APIUnderstanding of dynamic rule sets vs dynamic values
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
50
研究方向
Examine the JSS source code for handling dynamic rule sets, particularly the updatePath logic. Compare with dynamic values handling. The codesandbox provides a minimal reproduction. Investigate why null values do not trigger removal of previous styles. Look at related issue #1026 for context.