Frontmatter
| id | 9307 |
| title | Resolve naming clash for Icon/IconLink column iconCls |
| state | Closed |
| labels | bugai |
| assignees | tobiu |
| createdAt | Feb 26, 2026, 2:32 PM |
| updatedAt | Feb 26, 2026, 2:37 PM |
| githubUrl | https://github.com/neomjs/neo/issues/9307 |
| author | tobiu |
| commentsCount | 1 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Feb 26, 2026, 2:37 PM |
Resolve naming clash for Icon/IconLink column iconCls

tobiu
Feb 26, 2026, 2:36 PM
Input from Gemini 3.1 Pro:
✦ I have implemented the change across both the components (
Neo.component.Icon,Neo.component.IconLink) and the grid column definitions (Neo.grid.column.Icon,Neo.grid.column.IconLink), renaming the config fromiconClstocellIconCls. TheGridContainerinside DevIndex has also been updated. The changes have been pushed to thedevbranch.
tobiu assigned to @tobiu on Feb 26, 2026, 2:36 PM
tobiu closed this issue on Feb 26, 2026, 2:37 PM
Problem: When using
IconorIconLinkcolumn types in a grid, configuring aniconClsfor the cell icons inadvertently overwrites the column header's sorting arrow icon. This is because the grid container uses the column's configuration object to instantiate both the cell components and the header buttons.Proposed Solution: Introduce a new config
cellIconClsspecifically for the cell-level components, allowingiconClsto remain the domain of the header button (if desired).src/grid/column/IconLink.mjs: ReplaceiconClsconfig withcellIconCls. UpdateapplyRecordConfigsto mapcellIconClstoiconClsfor the underlyingNeo.component.IconLink.src/grid/column/Icon.mjs: ReplaceiconClsconfig withcellIconCls. UpdateapplyRecordConfigsto mapcellIconClstoiconClsfor the underlyingNeo.component.Icon.apps/devindex/view/home/GridContainer.mjs: Update configurations to usecellIconClsinstead oficonCls.