opendev/gerrit
Code Issues Proposed changes

ChangeScreen2: Show number of related changes in the tab title

Browse Source 
If the number of related changes is shown in the tab title a user can
immediately see that there are related changes even if another tab is
currently selected.

Change-Id: I8aa4ce6fea89fb4e05fbd4053f97095c06e96dcf
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2013-10-11 09:33:27 +02:00
committed by Edwin Kempin
parent 292593145e
commit 3bf14bf539
6 changed files with 27 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Constants.java
View File

@@ -27,5 +27,6 @@ interface Constants extends com.google.gwt.i18n.client.Constants {
String draft(); String draft();
String draftCommentsTooltip(); String draftCommentsTooltip();
String notAvailable();
String relatedChangesTooltip(); String relatedChangesTooltip();
} }

1
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Constants.properties
View File

@@ -10,4 +10,5 @@ author = Author / Committer
draft = (DRAFT) draft = (DRAFT)
draftCommentsTooltip = Draft comment(s) inside draftCommentsTooltip = Draft comment(s) inside
notAvailable = N/A
relatedChangesTooltip = Same branch changes connected by Git history relatedChangesTooltip = Same branch changes connected by Git history

20
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Messages.java Normal file
View File

@@ -0,0 +1,20 @@
// Copyright (C) 2013 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.gerrit.client.change;
public interface Messages extends com.google.gwt.i18n.client.Messages {
String relatedChanges(int count);
String relatedChanges(String count);
}

1
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Messages.properties Normal file
View File

@@ -0,0 +1 @@
relatedChanges = Related Changes ({0})

3
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/RelatedChanges.java
View File

@@ -127,11 +127,14 @@ class RelatedChanges extends Composite {
.get(new AsyncCallback<RelatedInfo>() { .get(new AsyncCallback<RelatedInfo>() {
@Override @Override
public void onSuccess(RelatedInfo result) { public void onSuccess(RelatedInfo result) {
tabPanel.getTabBar().setTabText(0, Resources.M.relatedChanges(result.changes().length()));
render(revision, result.changes()); render(revision, result.changes());
} }
@Override @Override
public void onFailure(Throwable err) { public void onFailure(Throwable err) {
tabPanel.getTabBar().setTabText(0,
Resources.M.relatedChanges(Resources.C.notAvailable()));
progress.setVisible(false); progress.setVisible(false);
scroll.setVisible(false); scroll.setVisible(false);
UIObject.setVisible(error, true); UIObject.setVisible(error, true);

1
gerrit-gwtui/src/main/java/com/google/gerrit/client/change/Resources.java
View File

@@ -22,6 +22,7 @@ import com.google.gwt.resources.client.ImageResource;
public interface Resources extends ClientBundle { public interface Resources extends ClientBundle {
public static final Resources I = GWT.create(Resources.class); public static final Resources I = GWT.create(Resources.class);
static final Constants C = GWT.create(Constants.class); static final Constants C = GWT.create(Constants.class);
static final Messages M = GWT.create(Messages.class);
@Source("star_open.png") ImageResource star_open(); @Source("star_open.png") ImageResource star_open();
@Source("star_filled.png") ImageResource star_filled(); @Source("star_filled.png") ImageResource star_filled();