Excel comment window not closing when mouse out/away from cell #998

Closed
opened 2018-02-16 12:03:26 +00:00 by cmuruganmsc · 2 comments
cmuruganmsc commented 2018-02-16 12:03:26 +00:00 (Migrated from github.com)

Hi, we are using xlsx@v0.12.1 plugin (https://www.npmjs.com/package/xlsx) for writing data in exported excel file also try to add cell comment. After export the data the cell comment added for applicable cells and when we hover on cell the comment window open/close working fine with Read Only mode. When we edit the download excel file from internet/my server. The comment window failed to close when away from cell.

Please let me know if we need to add any new configuration properties in write_comments_vml() method to avoiding comment window close issue and its get works as expected. Also attached screen shot and download file for reference.

Note: Please not down the excel comment working as expected in Read Only mode.
Excel-cell-comment-window-not-close.xlsx
excel-comment-window-not-close

Hi, we are using xlsx@v0.12.1 plugin (https://www.npmjs.com/package/xlsx) for writing data in exported excel file also try to add cell comment. After export the data the cell comment added for applicable cells and when we hover on cell the comment window open/close working fine with Read Only mode. When we edit the download excel file from internet/my server. The comment window failed to close when away from cell. Please let me know if we need to add any new configuration properties in write_comments_vml() method to avoiding comment window close issue and its get works as expected. Also attached screen shot and download file for reference. Note: Please not down the excel comment working as expected in Read Only mode. [Excel-cell-comment-window-not-close.xlsx](https://github.com/SheetJS/js-xlsx/files/1731219/Excel-cell-comment-window-not-close.xlsx) ![excel-comment-window-not-close](https://user-images.githubusercontent.com/5468492/36306834-51354e42-12ce-11e8-97e3-a0b1162a279e.png)
SheetJSDev commented 2018-05-20 18:13:42 +00:00 (Migrated from github.com)

That is controlled by the <x:Visible> tag and default shape css style. The fix, which we will push in the next version, will control both through a hidden property on the comment object.

That is controlled by the [`<x:Visible>` tag](https://github.com/SheetJS/js-xlsx/blob/master/bits/55_vml.js#L38) and [default shape css style](https://github.com/SheetJS/js-xlsx/blob/master/bits/55_vml.js#L22). The fix, which we will push in the next version, will control both through a `hidden` property on the comment object.
dylan-shao commented 2020-11-12 09:37:35 +00:00 (Migrated from github.com)

Hi, I am using 0.14 version, still not fixed here

Hi, I am using 0.14 version, still not fixed here
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sheetjs/sheetjs#998
No description provided.