假如有这样一个 NavBar 组件,被 50%以上的页面使用,下面 React 简略代码
interface NavbarProps {
BackAction?: ({ navigation, ...rest }) => JSX.Element
Content?: ({ title, ...rest }) => JSX.Element
Action?: ({ ...rest }) => JSX.Element
title?: string
}
const Navbar = (props:NavbarProps)=>{
const {BackAction = DefaultBackAction, Content = DefaultContent, Action = DefaultAction, title, } = props || {}
return <div className="flex flex-row">
<BackAction {...defaultBackActionProps}>
<Content {...{...defaultContentProps,title}}/>
<Action {...defaultActionProps}/>
</div>
}
// demo
return <Navbar title="title" />
现在你要给这个组件的 title 加一个标题为当前播放音乐主题色的功能,但是只有 1 个页面会用到这个功能
const color = useMusicThemeColor()
1.增加一个 isMusicTheme=true ,然后把 useMusicThemeColor 放在 NavBar 内部
<Navbar title="title" isMusicTheme={true} />
2.override Content ,把 useMusicThemeColor 放当前 page
const color = useMusicThemeColor()
...
<Navbar title="title" Content={({...rest,title})=>{
return <div {...rest} style={{color}}>
{title}
</div>
}} />
1
chenliangngng 33 天前
你可以把组件套一层壳,然后加上你要的功能,不当公共组件
|
2
shintendo 33 天前
不能让 title 可以接受 JSX.Element 吗
|
3
zhengfan2016 OP @shintendo 建议您阅读一下代码里 Content 组件,思考为什么 Content 组件会接收 Navbar 传过来的 title🤔
|
4
shintendo 33 天前
@zhengfan2016 确实,没仔细看。我选择加个 titleStyle
|
5
llwxi 33 天前
先用第二种实现,后续如果有两个组件使用到了,封装一个<MusicThemeColorText>{title}</MusicThemeColorText> 之类的组件?
|
6
zhengfan2016 OP |
7
zb1141920796 33 天前
图省事就用 1 ,图可扩展性强一点就用 2,
|
8
snowlee 32 天前
我会把 title 变成 reactNode, 相传啥就传啥
|
9
lisongeee 32 天前
没看懂你的操作,内部直接用 css 变量不行吗?
然后直接在目标页面给 body 设置 css 变量,离开后移除此变量 |
10
monologue520 32 天前
1.设 flag 控制,如果确定只有一个页面发生变化未尝不可,比较直观简单
2.增加了组件拓展性,控制当前页的颜色,还有其他的页的 title 默认颜色,可以抽离一个 Title 组件维护 color 属性 3.使用样式变量,比如 cssinjs 写法中「 const color = useMusicThemeColor()」传递一个 props 给 title 样式,最简单方便 我个人选 3 (用不着不爽,沟通一下就可以了) |
11
iOCZS 32 天前
要是既可以简单传标题,又可以自定义标题内容就好了。vue slot 那套还是比较合适这个想法的。
|