solex2006 / SELIProject

SELI Project
9 stars 17 forks source link

Authoring Tool menu #72

Open ecureuill opened 4 years ago

ecureuill commented 4 years ago

:warning: Wire-frames and live-demos are just for example purpose. They don't meet entirely this requirement.

:pencil2: Pencil icon marks specs that are waiting for definition


This requirement is about the redesign of Authoring Tool menu.

Index

  1. Redesign
    1. Button Style
  2. Feedback
  3. Bottom Menu
    1. Step navigation buttons
    2. Course creation buttons
    3. Feedback container
  4. Responsive Design
    1. Option A
      1. Medium screen
      2. small screen
    2. Option B
      1. Medium screen
      2. small screen

Currently version image

Redesign

image

Note: There is alternative options of redesign, check the wireframes and the live-demo

Play with the live demo to check the Resposive Design proposals and the Stepper feature. https://tsz10.csb.app/ The source is here

Buttons style

image

Feedback

Success Feedback

Suggestions and warnings Feedback

Process progress Feedback

Error Prevention

If user tries to navigate away (click to go to Home, close browser tab or enter a URL on browser) AND there is unsaved work

If user select to Close AND there is unsaved work

If user select to Publish

Step navigation buttons

Course creation buttons.

Stepper

The current version of the system has a stepper implemented. This proposal uses the same component, Material-UI Stepper

Step Status

[STEP LABEL], [STEP DESCRIPTION] and [TOTAL] are defined in #73

States

state icon color label triggered
Initial STEP ICON blue n/a default
Completed check icon green Completed All mandatory fields have been filled
Incomplete warn icon red Required Step Not all mandatory fields have been filled
Disabled theme-specific gray n/a n/a

[STEP ICON] is defined in #73

image

Responsive Design

NOTE: This wire-frames are just for example purpose. They are no meet entirely this requirement, for instance: Cancel button nos is Close

image

Option A

Large screen

image

Medium screen

vertical stepper image image

Small screen

smaller status text image

Option B

Medium screen (stepper hidden)

image

Small

smaller status text image

Smaller

image

Option C

Medium screen

image

Small

smaller step status text image

Smaller

image


RESOURCES

Material design

image image image image image

github-actions[bot] commented 4 years ago

This Feature is ready to be implemented.

ecureuill commented 4 years ago

@juantenesaca this specification is ready, the pencil on the top is just a legend, but as you see its not in use on the body of the specification, thus no pending here

juantenesaca commented 4 years ago

@ecureuill I was talking about the #74, this one we are going to finish when we will get something stable from the others, because we need to pass some props to modify the state of the step, e.g (Complete, Incomplete, etc)

caroltaus commented 4 years ago

image

caroltaus commented 4 years ago

(moved to #73) - [ ] When in Free Mode the steps are numbered incorrectly

5th step is shown as 6 of 7 image

7th step is shown as 8 of 7 image

ecureuill commented 4 years ago

Responsive design issue

Tested in a 13.3" screen (16:9 - 1366x768) image





ecureuill commented 4 years ago

Please, move to CODED only when this feature is fully implemented: bugs reported in comments and requirements in OP If you want that we test some partial solution, add a comment here with our @

github-actions[bot] commented 4 years ago

This feature have been implemented and is ready for Funcional Test. Please, test the feature.

github-actions[bot] commented 4 years ago

Functional Test failed! Please, review your solution.


There are some tasks still to be implemented


Hi! We noticed that in the requirement there were supposed to be three course creation buttons (Close, save and publish course). But, in the current platform there are "Save", "Publish" and "Cancel".

Requirement:

Captura de Tela 2020-10-13 às 16 51 18

Current website:

Captura de Tela 2020-10-13 às 16 53 06

While the close button was to exit the authoring tool, the cancel one is not doing that. Because of that, the user can't leave the page.

Was this change on purpose? Because we think a close button is good for the website usability.

CAndresH commented 4 years ago

Hello @rebeccamello and @caroltaus , I remember that some time ago, @juantenesaca talked to Camila about this button and it was considered to change it to the cancel button, this button was eliminated since it was not very functional, to exit a course you can do it using the platform menu

github-actions[bot] commented 4 years ago

This feature have been implemented and is ready for Funcional Test. Please, test the feature.

github-actions[bot] commented 4 years ago

Functional Test failed! Please, review your solution.


There are still tasks in the requirement to be done and in comment https://github.com/solex2006/SELIProject/issues/72#issuecomment-673642652


Steps to reproduce the error:

image

github-actions[bot] commented 4 years ago

This feature have been implemented and is ready for Funcional Test. Please, test the feature.

github-actions[bot] commented 4 years ago

Functional Test failed! Please, review your solution.


The requirement for these situations is still missing

image