Anonymitaet commented on code in PR #644:
URL: https://github.com/apache/pulsar-site/pull/644#discussion_r1266608941


##########
static/assets/broker-load-balancing-1.png:
##########


Review Comment:
   For reviewers:
   
   Here are some known issues of illustrations: 
   
   - They are not in a consistent style, and some of them are not clear enough. 
I tried to optimize them but failed. Previously I drew them in LucidChart, but 
we can't log into it suddenly. I've reported this issue and waiting for the 
response. I'll adjust these images once we have access to LucidChart.
   
   - Some of them are too big. I've tried many solutions (e.g., 
https://linuxhint.com/add-images-markdown-modify-image-size/ and 
https://stackoverflow.com/questions/14675913/changing-image-size-in-markdown), 
but none of them worked. I guess this is a limitation of Docusarus since [its 
official doc](https://docusaurus.io/docs/markdown-features/assets) does not 
even mention "resizing image". I'll optimize them once I find a feasible 
solution.
   



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to