akveo / akveo/nebular

Icon won't allow config on enabled tab of NbRouteTabsetComponent

未关闭
#2,337 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
TypeScript
星标
8.1k
派生
1.5k
PR 合并指标
30 天内没有已合并 PR

描述

### Issue type

**I'm submitting a ...** (check one with "x")

* [x] bug report
* [ ] feature request

### Issue description
According to https://github.com/akveo/nebular/pull/1935 (which closed https://github.com/akveo/nebular/issues/1323 ) , components which use icons should be accepting an icon config object (instead of just a string), however this commit https://github.com/akveo/nebular/pull/1935/commits/13d74cf79adcf77f4bf633e1559a315d9d59fad2 is missing the change on the `enabled` tab template, so it only works for disabled tabs.

The fix should be easy, just change the line:
https://github.com/akveo/nebular/blob/ac50daebc6b3cbae52f593bd2c878d7cacb94246/src/framework/theme/components/route-tabset/route-tabset.component.ts#L105

to bind to the `[config]` property instead of `[icon]`, as done on the `disabled` tab code a few lines above it on the specified commit:
https://github.com/akveo/nebular/blob/ac50daebc6b3cbae52f593bd2c878d7cacb94246/src/framework/theme/components/route-tabset/route-tabset.component.ts#L91

### Other information:

Tested on Angular 9 and nebular 5.0, but the problem is quite obvious that it's still there on `master`, and should be an easy fix (I really don't have time to make a pull request right now or I'd gladly do it)

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。