Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback: Milestone 3 #252

Open
wasdJens opened this issue Mar 31, 2020 · 1 comment · Fixed by #263
Open

Feedback: Milestone 3 #252

wasdJens opened this issue Mar 31, 2020 · 1 comment · Fixed by #263
Labels
epic Too big for a single user story (abstract) feedback Customer Feedback

Comments

@wasdJens
Copy link
Contributor

wasdJens commented Mar 31, 2020

Description - 1)
Half input validation around the password input in the register form
Screenshot
Screenshot 2020-03-31 at 09 15 02
Possible Solution


Description - 2)
Wrong label usernamename and the html placeholder displays display name
Screenshot
Screenshot 2020-03-31 at 09 15 13
Possible Solution


Description - 3)
Inside the cms user list table the creation date and last seen are timestamps instead of "human readable dates".
Screenshot
Screenshot 2020-03-31 at 10 01 17
Possible Solution
Missing new Date(timestamp) most likely


Description - 4)
Inside the cms use a different design approach when it comes to the "navigation" on the left side. From a user perspective it is not clear what is a "heading" and what is an navigation tab.

Screenshot
Screenshot 2020-03-31 at 10 01 37

Possible Solution

  • Heading in bold
  • Add an underline

For reference purposes take a look at our administration navigation

Screenshot 2020-03-31 at 10 55 19

Screenshot 2020-03-31 at 10 55 35


Description - 5)
Change button label adopt use something like confirm or apply.

Screenshot
Screenshot 2020-03-31 at 10 07 26

Possible Solution


Description - 6)
It is not quite clear what the due date is related to at the moment. It is above the post poll button but is split with a line below the Create your poll content.

IMO the due date is related to all content types? Maybe you are already resolving this with #227

Screenshot
Screenshot 2020-03-31 at 10 31 16

Possible Solution


Description - 7)
When using multiple options dont use radio buttons mozilla developer docs

Only one radio button in a given group can be selected at the same time

Screenshot
Screenshot 2020-03-31 at 10 38 44

Possible Solution

  • Instead use normal checkboxes when the multiple option is set to true

Description - 8)
Incorrect translation for the "warning" when trying to vote without choosing an option

Screenshot
Screenshot 2020-03-31 at 11 09 26

Possible Solution
ein => eine


Description - 9)
Wrong translation for votes when using german

Screenshot
Screenshot 2020-03-31 at 11 07 16

Possible Solution


Description - 10)
Clicking on show results on a poll I can not go back to cast my possible vote. This is confusing for the user because maybe he wants to see the results before casting its vote

Screenshot
Screen Recording 2020-03-31 at 11 15 18 mov

Possible Solution


Description - 11)
Directly login after registering - it should be possible that you are logged in after registering for the system

Screenshot

Possible Solution


Description - 12)
No logout button.

Screenshot

Possible Solution
Add a logout button to the menubar somewhere. Also create a possible route /logout that basically logs the user out when visit.


Description - 13)
Add loading spinners. When data is loading display some kind of loading spinners so that the user knows something is loading.

Screenshot

Possible Solution

@wasdJens wasdJens added feedback Customer Feedback epic Too big for a single user story (abstract) labels Mar 31, 2020
@DevPhilB DevPhilB linked a pull request Apr 2, 2020 that will close this issue
DevPhilB pushed a commit that referenced this issue Apr 3, 2020
* Update register page and directly login after registration

* Slightly improved CMS menue layout

* Update login page and add loading spinners to some pages

* Minor localization improvements

* Add logout redirect with /logout - delete local storage

* Update Board List and User List within CMS

* Add missing translations for CMS tables
@MistereoSC MistereoSC added this to the Milestone IV milestone Apr 13, 2020
@DevPhilB
Copy link
Contributor

DevPhilB commented Apr 22, 2020

1-9 & 11-13 are fixed.
12 moved to #254.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Too big for a single user story (abstract) feedback Customer Feedback
Development

Successfully merging a pull request may close this issue.

3 participants