[GitHub] [incubator-echarts] 100pah commented on issue #11458: pieselectchange event handler default data.seriesId value has extra whitespace character

[GitBox <gi...@apache.org>]

100pah commented on issue #11458: pieselectchange event handler default data.seriesId value has extra whitespace character
URL: https://github.com/apache/incubator-echarts/issues/11458#issuecomment-545277154
 
 
   The original intention of using special characters in auto-generated id is 
   (1) avoid conflicts in a simple way, like pissang said.
   (2) prevent users from using the auto-generated id. For some inner reasons, the auto-generated id has some that it composed with name and index. But users should not parse it because it might be changed one day.
   
   Of course this approach it is not a complete solution but it is simple. I prefer to suggest still keeping it until some strong reason or downside occur.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org