Skip to content

refactor: let SearchIcon inherit color from style context - #3735

Merged
UncleBill merged 2 commits into
developfrom
fix/inherit-color
Jul 23, 2021
Merged

refactor: let SearchIcon inherit color from style context#3735
UncleBill merged 2 commits into
developfrom
fix/inherit-color

Conversation

@UncleBill

@UncleBill UncleBill commented Jul 22, 2021

Copy link
Copy Markdown
Contributor

also remove @masknet/theme from icons' dependencies

also remove @masknet/theme from icons' dependencies
septs
septs previously approved these changes Jul 22, 2021
@UncleBill

Copy link
Copy Markdown
Contributor Author

I want to import an icon from @masknet/icons in @masknet/theme. @/icons already dependents @/theme, I don't want them to become a circular dependency to each other.

The reason I want an icon in @/theme is that a custom Snakbar I create in #3718 needs a spinning icon.

Reconsidering about the relationship between @/theme and @/icons, I think @/theme is a fundamental thing, it should be depended by any others, then we might need to move the components in @/theme to a new package to make @/theme more pure.

Would like to hear your opinion. @Jack-Works

"build": "node --es-module-specifier-resolution=node ./dist/utils/ssr.js"
},
"dependencies": {
"@masknet/theme": "workspace:*"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you remove this, please also remove it from tsconfig file.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any reference to @masknet/theme in packages/icon/tsconfig.json. What I miss?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh so that was a mistake when adding that dependency

@UncleBill
UncleBill merged commit af09658 into develop Jul 23, 2021
@UncleBill
UncleBill deleted the fix/inherit-color branch July 23, 2021 02:28
yanzhihong23 added a commit that referenced this pull request Jul 27, 2021
* develop:
  fix: absence account (#3752)
  feat: use swc to replace ts-loader (#3512)
  Revert "feat: enable es2020 syntax (#3750)" (#3758)
  feat: upgrade to holoflows-kit 0.9.0 (#3756)
  fix: type error (#3757)
  feat: enable es2020 syntax (#3750)
  refactor: remove ancient features (#3746)
  refactor: enhance robustness (#3664)
  chore: add isValidChainId (#3749)
  chore: update compatibility (#3747)
  fix: zerion with polygon & bsc (#3738)
  fix: the color of text placeholder (#3740)
  refactor: let SearchIcon inherit color from style context (#3735)
  chore: the default trending data provider (#3739)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants