Ported Sugarizer server's Tour: from Bootstrap to IntroJS

[rockharshitmaurya]

Hey, @llaske @NikhilM98 I have ported Sugarizer Server Dashboard Tour from Bootstrap to IntroJS. Please review it.

Refer issue: https://github.com/llaske/sugarizer-server/issues/363

[NikhilM98]

Great work!

I haven't reviewed the code yet, I'm waiting for workflow approval from @llaske.

However, there are a few things that can be improved.

• It will be better to make the target box lighter or transparent, there are some readability issues with this implementation.

• This view is completely covered.

• There is no "End" button at the end of the tour so user has to click on the "X" button to end the tour. I'll be nice if there is any possibility to add an "End" button.

• The tour restarts every time user loads any view even though the page is already visited.

[NikhilM98]

There are also some glitches in the tour which can be improved.

-

[NikhilM98]

@llaske can you approve the workflow run on this PR?

[rockharshitmaurya]

hey @NikhilM98, Thanks for the feedback,I will work on making the necessary changes and will update the issue as soon as possible.

[llaske]

I'm agree with @NikhilM98 remarks. Will be nice too to increase width of the title bar:

[NikhilM98]

@rockharshitmaurya can you also fix the failing CI checks?

[parteekcoder]

@rockharshitmaurya can you also fix the failing CI checks?

there are some lint error, to ease this I opened a PR #400 so that contributors will commit after removing lint errors as it is unnecessary efforts to push code and then resolve linting error, because sometime contributors forget to run command npm run lint before pushing code .It happens with me also

[rockharshitmaurya]

@rockharshitmaurya can you also fix the failing CI checks?

yeh sure @NikhilM98 sir , I'll be happy to fix that too

[rockharshitmaurya]

I'm agree with @NikhilM98 remarks. Will be nice too to increase width of the title bar:

sure @llaske sir, I'll Take Care of That

[rockharshitmaurya]

• There is no "End" button at the end of the tour so user has to click on the "X" button to end the tour. I'll be nice if there is any possibility to add an "End" button.

hey @llaske @NikhilM98 , I actually started working on this PR and I have a doubt. all the new tour of sugarizer which are using intro.js do not contain an "End" button, they only have an "X" button on the top right bar to end the tour. So, would it be alright to add an "End" button in the server tour, as it would be different from the others?

[llaske]

@rockharshitmaurya no you're right, no end button should be provided.