azure-webpubsub icon indicating copy to clipboard operation
azure-webpubsub copied to clipboard
verified publisher icon Azure

Keving/demosite/UI refresh

Open kevinguo-ed opened this issue 3 years ago • 4 comments

[What's changed?]

  • Refreshed the UI
  • Page for demos has an quick-to-navigate side bar
  • Added a Blog page
  • Added a Contact page

[What still needs to be done?]

  • The textual context is primary placeholders. I am writing content for them now.
  • Images are not optimized for performance

kevinguo-ed avatar Aug 24 '22 09:08 kevinguo-ed

should not make such a big PR should not contains place holders

wanlwanl avatar Aug 25 '22 10:08 wanlwanl

the most important problem is still there: https://github.com/Azure/azure-webpubsub/pull/446#discussion_r954770610

wanlwanl avatar Sep 21 '22 06:09 wanlwanl

Please find the screenshots of the pages needing review below.

home-web demo-web blog-web contact_us-web

home-mobile demo-mobile blog-mobile contact_us-mobile

kevinguo-ed avatar Sep 22 '22 08:09 kevinguo-ed

since this version is hurry to onboard, please fix the following main issues, other issues can be fix later:

  1. cookie banner looks like is not ready, need double check
  2. content is not ready
  3. comment and indent should be remove/update (simple to fix, so please also fix them)

wanlwanl avatar Sep 23 '22 08:09 wanlwanl

due to conflict issue, update in #453 #454 #455 #456 so close this PR

wanlwanl avatar Nov 02 '22 07:11 wanlwanl