You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by ah...@apache.org on 2017/04/05 04:37:19 UTC
zeppelin git commit: [ZEPPELIN-2339] Change 'see more' -> 'enable'
next to the older ver of interpreter type pkg
Repository: zeppelin
Updated Branches:
refs/heads/master 79a53129e -> 85c525200
[ZEPPELIN-2339] Change 'see more' -> 'enable' next to the older ver of interpreter type pkg
### What is this PR for?
Currently "see more" is placed next to the older version of INTERPRETER type of Helium pkg. But other type of Helium pkg(e.g. SPELL or VISUALIZATION) has "enable". And if we click "enable", can see the dialog for asking if you want to enable the package or not. So I changed INTERPRETER type pkg's "see more" to "enable" for the consistency in this patch.
### What type of PR is it?
Improvement
### What is the Jira issue?
[ZEPPELIN-2339](https://issues.apache.org/jira/browse/ZEPPELIN-2339)
### How should this be tested?
1. Run Zeppelin web under `zeppelin-web` with
```
$ yarn run dev
```
2. Go to Helium menu and click any interpreter type of package
3. Click `versions`
4. there should be "enable" not "see more"
### Screenshots (if appropriate)
- Before
- After
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: AhyoungRyu <fb...@hanmail.net>
Closes #2211 from AhyoungRyu/ZEPPELIN-2339 and squashes the following commits:
64597ec [AhyoungRyu] Change 'see more' -> 'enable' next to the older ver of intp type pkg
Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/85c52520
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/85c52520
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/85c52520
Branch: refs/heads/master
Commit: 85c5252006d2d4db47ff4404cf72a1b23a47ec01
Parents: 79a5312
Author: AhyoungRyu <fb...@hanmail.net>
Authored: Sat Apr 1 18:47:42 2017 +0900
Committer: ahyoungryu <ah...@apache.org>
Committed: Wed Apr 5 13:37:11 2017 +0900
----------------------------------------------------------------------
zeppelin-web/src/app/helium/helium.html | 7 -------
1 file changed, 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/zeppelin/blob/85c52520/zeppelin-web/src/app/helium/helium.html
----------------------------------------------------------------------
diff --git a/zeppelin-web/src/app/helium/helium.html b/zeppelin-web/src/app/helium/helium.html
index beb6191..fb604bf 100644
--- a/zeppelin-web/src/app/helium/helium.html
+++ b/zeppelin-web/src/app/helium/helium.html
@@ -131,16 +131,9 @@ limitations under the License.
ng-repeat="pkgSearchResult in pkgSearchResults[pkgSearchResult.pkg.name]">
{{pkgSearchResult.pkg.artifact}} -
<span ng-click="enable(pkgSearchResult.pkg.name, pkgSearchResult.pkg.artifact, pkgSearchResult.pkg.type, pkgSearchResult.pkg.groupId)"
- ng-if="pkgSearchResult.pkg.type !== 'INTERPRETER'"
style="margin-left:3px;cursor:pointer;text-decoration: underline;color:#3071a9">
enable
</span>
- <a target="_blank"
- ng-if="pkgSearchResult.pkg.type === 'INTERPRETER'"
- style="margin-left:3px;cursor:pointer;text-decoration: underline;color:#3071a9"
- href="http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22{{pkgSearchResult.pkg.artifact.split('@')[0]}}%22%20AND%20v%3A%22{{pkgSearchResult.pkg.artifact.split('@')[1]}}%22">
- see more
- </a>
</li>
</ul>
<div class="heliumPackageDescription" ng-bind-html="getDescriptionText(pkgSearchResult)"></div>