Found two bugs? (not sure)

Found two bugs? (not sure)

Posted by: horvat.f on 20 October 2023, 1:16 am EST

• 

  horvat.f

  • Post Options:
  • Link
    Copy
  • Jump To Answer

  Posted 20 October 2023, 1:16 am EST - Updated 20 October 2023, 1:18 am EST

  Hi!
  
  First bug (by testing)

  • Calling the Print Function does not take care of the fact that there’s a chart on the page, BUT it appears if you make a manual selection including columns where the chart is
    
    *Bypass: *make a selection manually then print with the “print selection”

  Second bug

  • If you name a sheet with spaces (for example “My new sheet”), the Excel export breaks the link between the selction and the chart
    
    *Bypass: *Name my sheets without any space
• 

  ankit.kumar

  

  • Post Options:
  • Link
    Copy

  Posted 22 October 2023, 11:55 pm EST

  Hi,

  Issue1: I was unable to reproduce the issue at my end with the latest version of SpreadJS V16.2.4. I created a sample with the chart and when calling the print() method, the print shows the chart.
  
  Sample: https://jscodemine.grapecity.com/share/g0kMX5BTG0OZhA0sypiyFw/?defaultOpen={"OpenedFileName"%3A["%2Findex.html"%2C"%2Fapp.js"]%2C"ActiveFile"%3A"%2Fapp.js"}

  Could you kindly share us a minimal working sample replicating the issue? You may also modify the above sample and share it with us replicating the issue.

  Issue2: I was also unable to replicate the issue with the latest version. Kindly refer to the following sample.

  Sample: https://jscodemine.grapecity.com/share/hCgpA6uLFUemhEiFBNFXkg/?defaultOpen={"OpenedFileName"%3A["%2Findex.html"%2C"%2Fapp.js"]%2C"ActiveFile"%3A"%2Fapp.js"}

  Could you kindly also share the sample replicating the issue?

  Also, kindly let me know if I am missing any steps in order to reproduce the issue.

  Regards,

  Ankit

• 

  horvat.f

  • Post Options:
  • Link
    Copy

  Posted 23 October 2023, 12:26 am EST

  I have badly described my context because SpreadJS is not used directly. I actually work with a library witch embeds yours. I’m quite sure this integration is not totally up-to-date.

  I’ve tried your test and it seems to work.

  Thank you, i’m actually communicating with the team who’s giving this library!

• 

  horvat.f

  • Post Options:
  • Link
    Copy

  Posted 23 October 2023, 11:41 pm EST

  I’ve found the source of the second “bug”. the length of the sheet’s name is about 30/32.
  
  I’ve tested on your sample, and it’s make the same issue: the Excel export doesn’t run well. under 30 car. no issue at all!

• 

  ankit.kumar

  

  • Post Options:
  • Link
    Copy

  Posted 24 October 2023, 3:42 pm EST

  Hi,

  The Microsoft Excel has set the character limit for excel files to be 31 characters whereas SpreadJS doesn’t have such limit. You may refer to the following stackoverflow post for more info: https://stackoverflow.com/questions/3681868/is-there-a-limit-on-an-excel-worksheets-name-length

  Yes, if the sheetName is above 31 characters, it will cause an issue when opening the Excel files in SpreadJS.

  You could use the SheetNameChanging Event to check for the sheet name’s length and you may cancel the operation.

  SheetNameChanging Event: https://www.grapecity.com/spreadjs/api/v15/classes/GC.Spread.Sheets.Events#sheetnamechanging

  Regards,

  Ankit