pik-gane/vodle

simplify first-time user's start

mensch72 opened this issue · 5 comments

When being invited to a poll, a first-time vodle user should be presented with the poll's welcome page immediately rather than with the login page. Such a user should be able to use the main parts of the voting gui without having to log in first. Only for transmitting their waps they should be required to log in.

In particular, the voting gui (=poll page) for a non-logged-in invited user should:

  • disable the "live results", "agreement details", "add option", and "delegate" buttons
  • present a privacy agreement checkbox as an overlay at the bottom of the page
  • disable sliders as long as the privacy agreement checkbox is not checked
  • show a banner at the top explaining that assigned waps are only getting effective after logging in, and presenting a two login buttons: "guest login" and "personal login"
  • show the login page as an overlay when clicking these buttons
  • do not submit any waps to the poll database before user is logged in

In addition, the login page could also be shortened by skipping the language selection whenever the browser specifies a preferred language that is supported by vodle.

Hello! Can I work on this one? 🚀

Dear @catalin-casuneanu , of course! Thank you so much!

It will be some work I guess though...

The most relevant code parts are here (*.page.ts and *.page.html are relevant):

Some more things:

  • The checkbox, text, and link for the privacy statement in the overlay at the bottom should be the same as on the login page.
  • The login page for returning users should stay as it is, but we may be able to make the same code also work in an overlay to the poll page for first-time users, to avoid redundant code?

Thank you for the information!
I'm experiencing some errors on setup, because I'm on a M1 Macbook, so I will uninstall node-sass (which is deprecated anyway) and install sass to fix it.
I can then switch back before I create a PR, if you don't want to switch to sass instead of node-sass permanently.
More details on the error here

Good to know, hopefully that won't break any dependencies.

Happy holidays, @catalin-casuneanu ! Are you making progress? Do you need anything from me?