1502shivam-singh commented on issue #450: URL: https://github.com/apache/apisix-website/issues/450#issuecomment-899813306
> Nice design @1502shivam-singh! Do you mind creating a markdown version of all the text in the new design and create a not-merge PR for it. I think the text part review could be easier with that. Woh, really nice suggestion, you would then be able to provide changes in a better way using github UI. Excellent idea, on it. Also to avoid fiasco with the actual apisix-website repo, I would make a PR on my own repo, and will share link here. > 1. the right part of the architecture graph seems a little unbalanced, since the all contents are in the upper part. Also we might could change `APISIX has dynamic routing and plug-in hot loading` to `APISIX has dynamic routing and hot-loading plugins` to keep the "adj.-n." format Okay, i guess pulling the etcd cluster little bit down to align with bottom edge APISIX block should do. Agreed with the latter. > 2. Seem missing a period here: "It’s opensource and ever evolving(.) feel free to contribute". And also we need a screenshot of dashboard in higher resolution Noted. I would say a comma maybe, but yeah i got you. On the screenshot, yeah on it, I guess compression did hurt it too much. > 3. The graph of `User Flexible` seems not correct. The description of it seems related to low code (the graph used in "Pluginised workflow"). Hmm, i was actually referencing the dual ability of code and drag and drop of APISIX. You are right, this one actually needs copywriting changes, like to - `The APISIX dashboard is flexible to the user demand, providing option to write code to create custom modules as per your needs, alongside the existing no-code toolchain` > 4. the description here is not so correct. The high performance should come from radixtree-route-matching and also etcd, not from plugins. Also, built-in plugins and json schema is more related to convenience so apisix could be used out-of-box. The perspective I needed, thanks. Okay, need to modify this one. How about - `Apache APISIX uses radixtree-route-matching and etcd under the hood to provide you the ability to create high speed synchronized systems. From routing to built-in plugins, all these are designed and implemented to be uber performant with the minimum latency possible.` > 5. maybe we could have a button for the `check us out` or people might not regard it as clickable Oops missed this. Earlier I thought of a red link as usually seen elsewhere, but yeah a button seems more viable Thanks for the detailed review, will share PR link here. -- 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]
