Faas: Proposal: Enable "Remove function" in FaaS UI

Created on 21 Aug 2017  路  19Comments  路  Source: openfaas/faas

Issue

It is not possible to delete functions via the FaaS UI.

Proposed fix

Add an Angular card with a delete button below the Invoke function card. See this comment for some UX direction on implementation.

enhancement help wanted

Most helpful comment

Can we move forward with the last suggestion - but with a caveat:

  • Add the contextual menu top right
  • Keep invoke where it is and revisit after adding actions drop-down menu

All 19 comments

@johnmccabe Please can you put together screenshot in paint or whatever works and attach?

delete_idea

@alexellis this good for you?

While we've got this page open do we want to change FaaS to OpenFaaS in the top left too?

@johnmccabe that is too big and easy to hit. Perhaps this should be on a pop-up dialog via a "..." break-out.

@rgee0 good point, can you put that suggestion on issue #123 and ping @glyif?

dropdown

@alexellis with a dropdown, the wide delete button in a card seems like the more material-ly one to me fwiw, very subjective ofc :)

we can also do something like this so peeps don't accidentally delete

screen shot 2017-08-21 at 2 11 15 pm

If you want to mix both you could but it on the row with Invoke, but have the pop-up for confirming. I think the huge red button in the first screenshot could draw the eye too much and it'll be hard to find when you invoke a function (i.e. it will be pushed very far down the screen)

The PWD site has a much more subtle delete button:

screen shot 2017-08-21 at 10 15 33 pm

And with an ellipsis, looks cleaner when unselected..

dropdown

and selected...

dropdown

@alexellis I've come round to your suggestion here, think this is the nicest.

I'd say the delete button needs to be away from the function output. Top right feels better to me.

Hi, my thoughts on this issue:

  • I agree with the dropdown solution, that let delete option appears
  • I would place all action buttons at the same place, it makes more sense (invoke, actions, and maybe more in the future) at top right
  • Use label and/or pictograms could helps user to understand the UI (e.g.: http://fontawesome.io) - 鈿狅笍 Not everywhere of course, but it can be self-explanatory in some situations

An attached screenshot with what I think quite appropriate (did not saw @johnmccabe proposition at first, that is quite similar):
screenshot 2017-08-23 09 40 09

  • The delete button is in a dropdown as @alexellis suggested

Can we move forward with the last suggestion - but with a caveat:

  • Add the contextual menu top right
  • Keep invoke where it is and revisit after adding actions drop-down menu

@fabienheureux - how do you feel about implementing the UI? cc @aafrey

@fabienheureux ping 馃彄

@alexellis sure, I will work on this next week 馃憤

func_deletion

Hi, I didn't realize this thread and was thinking about deletion, too. I basically agree with the final suggestions but how about making the "ACTIONS" button an ellipsis icon like @rgee0 suggests?

Here is a sample I was making. But including the above suggestions, the trash icon would become an ellipsis and also, it would become a dropdown.

@fabienheureux I could have a look at this today or tomorrow (JST timezone) so feel free to ping me if I can help!

I like this final gif - @kenfdev if you want to commit this in a PR I think we could just merge it and close this issue.

@alexellis No problem. I think I can implement it tonight (GMT+9)

Merged. Thank you to everyone on the thread for your contribution and to @kenfdev for the PR.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

edouardkleinhans picture edouardkleinhans  路  8Comments

hotjunfeng picture hotjunfeng  路  5Comments

ndarilek picture ndarilek  路  4Comments

alexellis picture alexellis  路  5Comments

derailed picture derailed  路  6Comments