Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
Current version : [pull#3226](https://github.com/apache/zeppelin/pull/3226)
---
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@mebelousov Got it. Thanks
---
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@jongyoul status is ready to merge to master =)
I hope I will merge this PR to our fork of branch-0.8 and will confirm that
works fine.
This needs 1-2 months.
---
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2708
Can someone share the status of this issue?
---
Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/2708
Fixed merge conflicts
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@prabhjyotsingh
Add block editing of paragraphs.
---
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
Tested it locally, and front-end changes looks OK.
In this case, I agree popup is not user-friendly. Also since we are
blocking (or temporarily disabling) Paragraphs Control, should we
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@prabhjyotsingh @r-kamath
could you help with the review?
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@zjffdu
I meant that the popup is not user friendly. The user sees a clickable
elements, clicks and the popup will be displayed
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@tinkoff-dwh I am not saying popup from backend. I mean backend send error
message to frontend, and frontend pop up this error message.
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@zjffdu
i think it is not user friendly interface (popups from backend)
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@tinkoff-dwh I am not frontend expert. Just going through the PR and have a
few thoughts:
1. The changes in frontend seems a little large. What if we add new UI
control ? Do we also need
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2708
yes, I think
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@felixcheung
What do you say about this?
![capture3](https://user-images.githubusercontent.com/25951039/34248134-ef41448e-e645-11e7-9678-66dfec98f204.PNG)
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2708
From a UX usability point of view it seems better to disable a control
instead of hiding it.
Would love others thought on that as well.
---
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2708
@felixcheung
I fixed it
---
16 matches
Mail list logo