Skip to content

Conversation

@vgmartinez
Copy link
Contributor

add notification when saved paragraph

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is Notification defined?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is defined in bower_components/angular-ui-notification/dist/angular-ui-notification.min.js

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! I can't find it under zeppelin-web\bower_components\angular-ui-notification folder. am I missing something?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@djoelz : Run the build then it will find it.
OR run bower install in zeppelin-web

@Leemoonsoo
Copy link
Member

Thanks @vgmartinez for adding notification.

I think notification is going to be useful, in general. This branch notify 'Saved' message when code is autosaved by timer. I'm not sure about this particular notification, because

  • Saving code already have visual indicator. (color of left vertical bar)
  • Other actions (Run, etc) also saves code. Notifying only when autosaved by timer may confuse user, that other actions does not save the code.

By the way, here's a screenshot, how it looks
image

@Madhuka
Copy link
Contributor

Madhuka commented Aug 19, 2015

This is good. We can used notification for showing data validation messages.

@vgmartinez
Copy link
Contributor Author

hi @Leemoonsoo ,
makes sense, sorry they had not seen the display ... but I think we can use notifications in other actions such as deleting a paragraph ... as they see?

@Leemoonsoo
Copy link
Member

Yes, notification would be useful for many cases. (connection error, etc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants