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

Clicking Stop Work Did Not Stop Work #2032

Closed
PixelantDesign opened this issue Aug 20, 2018 · 22 comments
Closed

Clicking Stop Work Did Not Stop Work #2032

PixelantDesign opened this issue Aug 20, 2018 · 22 comments
Labels
bug This is something that isn't working as intended.

Comments

@PixelantDesign
Copy link
Contributor

Describe the bug
@darkdarkdragon clicked stop work on an issue, and work was not stopped, or Gitcoin Bot did not log the stop action on the Github thread.

This is the issue:
#1769 (comment)

@mbeacom
Copy link
Contributor

mbeacom commented Aug 20, 2018

@PixelantDesign Can you start moving these bug issues into the Triage Bugs project and labeling them with the bug label so we can track them on the bug board?

@darkdarkdragon
Copy link
Contributor

I've clicked 'stop work', got notification 'Thanks for letting us know', so I've assumed it was stopped.
But later I've found out that it wasn't stopped.

@PixelantDesign PixelantDesign added the bug This is something that isn't working as intended. label Aug 20, 2018
@PixelantDesign
Copy link
Contributor Author

@mbeacom done

@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.1 ETH (27.76 USD @ $277.56/ETH) attached to it.

@pinkiebell
Copy link
Contributor

Maybe we shouldn't show a success banner before we have the result from the api? 😄
We should show a 'please hold the line'/ visual feedback animation instead,
until we actually get the api results. 👍

@pinkiebell
Copy link
Contributor

Something like that, maybe ? @PixelantDesign

out

@owocki
Copy link
Contributor

owocki commented Aug 22, 2018

i like this visually

@pinkiebell
Copy link
Contributor

@darkdarkdragon
Did you immediately closed the window after reading the notification?
Or better q: Did you seen two of these ''Thanks for letting us know..'?

@darkdarkdragon
Copy link
Contributor

@pinkiebell I don't remember, it was some time ago. Probably yes, I've closed window right after seeing 'thanks...'

@PixelantDesign
Copy link
Contributor Author

@pinkiebell
If I remember correctly, Gitcoin Bot did not comment on the ticket when @darkdarkdragon clicked Stop Work.

I think the green confirmation works as a cute to let users know that work has been stopped...but I'm not sure we need the big glow.

@pinkiebell
Copy link
Contributor

@PixelantDesign
The issue the way I see it is the following:
On start/stop work; (after submit) we display the green banner before we even make the API calls.
I think this is because of good faith and to have immediate UI feedback.
However, if the user now closes the tab/window it might happen that the request was still pending/not done.

This is a user experience issue, if I understand correctly.
I may also be completely wrong 😆

@thelostone-mc
Copy link
Member

thelostone-mc commented Aug 23, 2018

@pinkiebell Ah yeah that's on me!
The success banner should turn up only after getting a successful response from the API.
The waiting screen does seem to fix that

@PixelantDesign without the color it looks like this:

screen shot 2018-08-23 at 5 48 57 pm

It looks dull ! the blue looks too bright but it's subtle so I'm in 2 minds

@PixelantDesign
Copy link
Contributor Author

Thanks for looking at this @thelostone-mc @pinkiebell
Could we try this color #3E00FF with a 30% opacity behind the animated helmet?

@thelostone-mc
Copy link
Member

Not sure which one you meant by this is how it would look

screen shot 2018-08-23 at 6 57 51 pm

screen shot 2018-08-23 at 6 59 25 pm

@PixelantDesign
Copy link
Contributor Author

Would it start with the top one and then fade out? Hopefully this doesn't stay up long, ya?

@thelostone-mc
Copy link
Member

thelostone-mc commented Aug 23, 2018

@PixelantDesign mind throwing in a design to make things clearer ?
Merging it now -> and i'll update it once I see the design ^_^

@pinkiebell
Copy link
Contributor

@thelostone-mc @PixelantDesign
please try with: rgb(11,220,230) as box-shadow color 👍

showBusyOverlay();document.querySelector('.busyOverlay').style['box-shadow']='0 0 100em 0 inset rgb(11,220,230)'

screenshot 2018-08-23 at 19 42 08

@thelostone-mc
Copy link
Member

Deferring it to @PixelantDesign ! closing this out for now ^_^

@PixelantDesign
Copy link
Contributor Author

@pinkiebell could you click start work and submit work on this so that we can pay you?

@gitcoinbot
Copy link
Member

gitcoinbot commented Aug 27, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 11 months, 4 weeks from now.
Please review their action plans below:

  1. pinkiebell has started work.

    (* *)
    /.\
    (/ )
    (_ _)

Learn more on the Gitcoin Issue Details page.

@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 0.1 ETH (27.82 USD @ $278.22/ETH) has been submitted by:

  1. @pinkiebell

@owocki please take a look at the submitted work:


@gitcoinbot
Copy link
Member

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 0.1 ETH (27.82 USD @ $278.22/ETH) attached to this issue has been approved & issued to @pinkiebell.

vlad2689 pushed a commit to vlad2689/web that referenced this issue Aug 27, 2018
Full-page loading animation to let the user know something is happening.

Fixes/Related gitcoinco#2032
vlad2689 pushed a commit to vlad2689/web that referenced this issue Sep 15, 2018
Full-page loading animation to let the user know something is happening.

Fixes/Related gitcoinco#2032
vlad2689 pushed a commit to vlad2689/web that referenced this issue Sep 15, 2018
Full-page loading animation to let the user know something is happening.

Fixes/Related gitcoinco#2032
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is something that isn't working as intended.
Projects
None yet
Development

No branches or pull requests

7 participants