@billkamm
I suggest you add some buttons just for demo case. When clicking the button, it can mimic what action will do, etc. e.g., showing and hiding images, showing and hiding highlights and so on.
When you reproduce your issues by some specific steps, you can record all such steps one by one. For example, click button 1, switch to sheet 2, click button2, switch to sheet 3, and so on. Then we can help you to analyze your scenario to figure your issue out.
I noticed that we can no longer select images or shapes if the mode is âreadâ instead of âeditâ. Is there a way we would be able to select images/shapes while the mode is âreadâ? We want the workbook to be read only, but our users want to be able to right-click on images and shapes to open a context menu to allow them to apply highlights
@billkamm
sorry for the inconvenience
we will support to show right click context menu in read mode.
We have opened the following new ticket(s) in our internal issue tracking system and will deliver their fixes according to the terms mentioned in Free Support Policies.
Issue ID(s): CELLSGRIDJS-852
You can obtain Paid Support Services if you need support on a priority basis, along with the direct access to our Paid Support management team.
Upload Files Easily | Free File Upload and Sharing Up To 10GB Here is the project with the show/hide buttons. You can use the same Excel file as above.
Notice how after the worksheet loads and isImageOk is true and you click the Toggle Images button you get an âUncaught TypeError: Cannot read properties of undefined (reading ârequestRenderAllâ)â
When weâve tried to not remove all highlights and redraw them weâve run into issues with the show/hide highlights and transparency toggles not rendering the highlights correctly as we switch between the sheets of the worksheet. Sometimes highlights that should be hidden remain and sometimes highlights that should be showing do not appear. Weâve had similar issues with the images. A lot of this code we have in here is an attempt to get the features of these buttons to work as we change worksheets.
@billkamm,
Thanks for your details. We will evaluate your issue further and give feedback here later.
Thanks for the sample project.
We need to investigate your issue in details. We have opened the following new ticket(s) in our internal issue tracking system and will deliver their fixes according to the terms mentioned in Free Support Policies.
Issue ID(s): CELLSGRIDJS-857
You can obtain Paid Support Services if you need support on a priority basis, along with the direct access to our Paid Support management team.
@billkamm
Below is the updated index.html file which may work as per your expectation.
Please have a pre-review.
index.zip (5.5 KB)
You need to update to v23.7 version. We now add new APIs: xs.sheet.clearHighlights and xs.sheet.updateHighlightStyle and a new event sheet-loaded. We just simply use the addhighlight_in_sheet method instead of very complicated method drawRedactions for adding highlight process.
We use shape.setVisible method (which will work after image is loaded) instead of the visiblefilter (which will work before image is loaded from server).
We will soon publish the v23.7 release may be tomorrow.
Stay tuned!
The issues you have found earlier (filed as CELLSGRIDJS-852) have been fixed in this update. This message was posted using Bugs notification tool by johnson.shi
@billkamm
now you can update to our latest version v23.7.0. ,⌠wait for your feedback.
the latest dll
the latest js
After implementing these changes in our project Iâve run into an issue where xs.isLoaded() sometimes gets stuck never being true, so the setTimeout features in some of these methods end up in an infinite loop
const tryWaitImageReadyDoSetVisible = () => {
if (!xs.isLoaded()) {
setTimeout(tryWaitImageReadyDoSetVisible, 500);
} else {
setAllImageShapeVisibleStatus();
}
I havenât been able to an exact steps to reproduce to send you an example yet, but it seems to happen after I set one of these such as hide highlights or add transparency and then switch between sheets and try again
Thank you for your feedback. We will be looking into it and getting back to you soon. Additionally, we would appreciate it if you could provide us with a sample project/example and detailed steps to reproduce the issue on our end. This will help us evaluate the issue precisely and find a solution quickly.
@billkamm I also had an issue with loading state in 23.7.0. You may want to check the repro steps at Images are rendered in incorrect location if sheet is scrolled before image is loaded (gridjs-spreadsheet) - #19 by jrf.att to see if itâs the same problem.
@jrf.att
Weâve fixed this issue.It seems the issue has the same root cause from Images are rendered in incorrect location if sheet is scrolled before image is loaded (gridjs-spreadsheet)
We will send out new release version soon in this week.
In read only mode we canât access the text inside of a textbox object. Our application allows users to highlight text inside of a textbox object, but doesnât want users to be able to delete or edit the text. Would it be possible to make read only mode the way it was where you could select objects and the text within without being able to delete or edit objects or the data in the sheets?
To simplify I would like to be able to select text within a textbox object while in read only mode
it would be nice to still have the selection indicators on the images and shapes as well and being able to move them still like before would also be a plus
Thanks for sharing details about your new requirements. We will evaluate your needs and get back to you with details.
@billkamm
We will support select text within a textbox object while in read only mode.
We will support select on images/shapes in read only mode.(for move operation we donât think it is suitable in read only mode)
We have opened the following new ticket(s) in our internal issue tracking system and will deliver their fixes according to the terms mentioned in Free Support Policies.
Issue ID(s): CELLSGRIDJS-897,CELLSGRIDJS-898
You can obtain Paid Support Services if you need support on a priority basis, along with the direct access to our Paid Support management team.
1 Like