Skip to content

https://app.doboard.com/1/task/61019 - #141

Merged
veronika-tseleva-cleantalk merged 1 commit into
devfrom
fix_61019
Nov 9, 2026
Merged

veronika-tseleva-cleantalk merged 1 commit into
devfrom
fix_61019

Conversation

@veronika-tseleva-cleantalk

Copy link
Copy Markdown
Contributor

Copilot AI balanced review requested due to automatic review settings November 9, 2026 16:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

In-place refreshes leave the header’s comment count stale; update it from templateVariables.amountOfComments and regenerate the bundled widget.

1 open issue
What changed in this PR

This PR improves the doBoard widget’s task-header display, menu layout, and maximize behavior.

Changes:

  • Adds title truncation and tooltips while updating the title separately from the message count.
  • Allows long menu links to wrap and gives sticky footers shared styling.
  • Replaces repeated maximize listener registration with a single click handler and refreshes the distribution bundle.
File Description
styles/​doboard-widget.css Adds title, link-wrapping, and menu-footer styles.
js/​src/​widget.js Updates title text and tooltips; adjusts maximize handling.
js/​src/​main.js Allows title attributes on sanitized spans.
js/​src/​loaders/​SpotFixTemplatesLoader.js Updates title markup and task-menu layout.
dist/​doboard-widget-bundle.js Includes the updated widget logic, templates, and styles.

Review effort: Balanced

Comment thread js/src/widget.js
Comment on lines +1696 to +1699
const issueTitleElement = document.querySelector('.doboard_task_widget-issue-title-text');
if (issueTitleElement) {
issueTitleElement.innerText = ksesFilter(taskName);
issueTitleElement.innerText = ksesFilter(taskName || '');
issueTitleElement.title = issueTitleElement.innerText;
@veronika-tseleva-cleantalk
veronika-tseleva-cleantalk merged commit 0ab50ff into dev Nov 9, 2026
1 check passed
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.

3 participants