Skip to content

Show host description in Tiles and Time view#219

Merged
TheWitness merged 4 commits into
Cacti:developfrom
godmafiaze:fix-218-tilesadt-show-hostname
Jul 9, 2026
Merged

Show host description in Tiles and Time view#219
TheWitness merged 4 commits into
Cacti:developfrom
godmafiaze:fix-218-tilesadt-show-hostname

Conversation

@godmafiaze

Copy link
Copy Markdown
Contributor

This PR updates the Monitor plugin Tiles & Time view to display the host description together with uptime/time-in-state.

Previously, renderHostTilesadt() displayed only the device icon and uptime/time-in-state value, which caused devices to appear without visible names in the Tiles & Time view.

This change adds the escaped host description using title_trim() before the uptime/time-in-state value.

Fixes #218

Tested:

  • Verified Tiles & Time displays host description and uptime/time-in-state
  • Verified the Monitor page renders correctly after the change
  • Verified git diff --check returns no whitespace errors

@TheWitness

Copy link
Copy Markdown
Member

In the future @godmafiaze, when you add a feature like this, make sure you update the CHANGELOG.md

@TheWitness

Copy link
Copy Markdown
Member

Appreciate your support!

@TheWitness TheWitness merged commit 6c08178 into Cacti:develop Jul 9, 2026
4 checks passed
@TheWitness

Copy link
Copy Markdown
Member

This one created a bit of a problem. View it in some of the other themes and icon sizes. Does not look good. I think a Checkbox is a better approach, but we need to ensure that they are block objects so that the wrapping works.

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.

Tiles & Time view does not display host description

2 participants