Skip to content
This repository has been archived by the owner on Dec 10, 2019. It is now read-only.

Error when creating a node without selecting a node type #140

Closed
Hammster opened this issue Apr 8, 2017 · 12 comments
Closed

Error when creating a node without selecting a node type #140

Hammster opened this issue Apr 8, 2017 · 12 comments

Comments

@Hammster
Copy link
Collaborator

Hammster commented Apr 8, 2017

When creating a new node and no type is selected you can still press Create, which creates a error.

This should be handled or the Type selection radio should have a default selected. Also a little UX improvement 💪

@jccguimaraes
Copy link
Owner

did this help out?

if (!type) { return false; }

@Hammster
Copy link
Collaborator Author

Hammster commented Apr 8, 2017

Yep, but UX wise defaulting would be better. Or at least provide a small alert, so that the user knows what he forgot to set.

@jccguimaraes jccguimaraes modified the milestone: 1.0.x Apr 9, 2017
@jccguimaraes
Copy link
Owner

so you think it's best to make it default like to project?

@Hammster
Copy link
Collaborator Author

Hammster commented Apr 9, 2017

correct. then you have one click less for the default scenario.
Btw did you not had create project and create group in the context menu? Was there a reason to add the layer of the interface in between there? Or did you plan future use of the pane for a later expansion?

Sorry if I am bit nitpicky here 😸

@jccguimaraes
Copy link
Owner

hey @Hammster, sorry for late implementation. It's done now.

I didn't understand the last part "Was there..."

@Hammster
Copy link
Collaborator Author

Hammster commented Jun 13, 2017

Currently the steps are as followed

Right Click -> Create New -> Select the Type -> Fill Information -> Create

Before it was a little bit shorter

Right Click -> Create "Type" -> Fill Information -> Create

Was there a reason to do this ?

@jccguimaraes
Copy link
Owner

solo option for not having 2 menu items instead of just one... :D

@Hammster
Copy link
Collaborator Author

Hammster commented Jun 13, 2017

UX wise i would prefere to have two options, it feels a little bit artificial :P Also no worries for the late responses, i am also stuck in a pile of work 🔨

@jccguimaraes
Copy link
Owner

will try to add both then

@jccguimaraes jccguimaraes self-assigned this Jun 13, 2017
@jccguimaraes
Copy link
Owner

done! will ship on next release. Taking the time to close some others I hope

@jccguimaraes
Copy link
Owner

please validate this issue!

@Hammster
Copy link
Collaborator Author

Great work, nothing to complain about 😄👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants