Skip to content

Commit

Permalink
Merge pull request #8942 from infor-design/NG1678-Module-Nav-Collapsed
Browse files Browse the repository at this point in the history
NG1678 - ModuleNav Collapsed Tooltip
  • Loading branch information
tmcconechy authored Aug 6, 2024
2 parents 56010a9 + 2274959 commit 1c89932
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions docs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@

- `[Datagrid]` Optimize the initial loading of datagrids with filterable setting. ([#8935](https://github.com/infor-design/enterprise-ng/issues/8935))
- `[Datagrid]` Remove modification in calculateTextWidth that had incorrect selectors. ([#8938](https://github.com/infor-design/enterprise/issues/8938))
- `[ModuleNav]` Fixed a bug where the tooltip was not showing when collapsed by default. ([NG#1678](https://github.com/infor-design/enterprise-ng/issues/1678))

## v4.98.0

Expand Down
2 changes: 1 addition & 1 deletion src/components/module-nav/module-nav.js
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ ModuleNav.prototype = {
this.accordionAPI.updated(newSettings);

// Build tooltips on top-level accordion headers in collapsed mode
const headers = this.accordionEl.querySelectorAll('.accordion-section > .accordion-header, .accordion-section > soho-module-nav-settings > .accordion-header');
const headers = this.accordionEl.querySelectorAll('.accordion-section > .module-nav-switcher, .accordion-section > .accordion-header, .accordion-section > soho-module-nav-settings > .accordion-header');
if (headers.length) {
[...headers].forEach((header) => {
configureNavItemTooltip(header, this.settings.displayMode);
Expand Down

0 comments on commit 1c89932

Please sign in to comment.