Skip to content

Commit 1a3f7f1

Browse files
authored
Fix Eclipse UI issue (#3549)
* Fix eclipse will auto switch login method when browse auth file * Fix save and discard button be visiable without change
1 parent 5110549 commit 1a3f7f1

File tree

2 files changed

+11
-2
lines changed
  • PluginsAndFeatures/azure-toolkit-for-eclipse

2 files changed

+11
-2
lines changed

PluginsAndFeatures/azure-toolkit-for-eclipse/com.microsoft.azuretools.azureexplorer/src/com/microsoft/azuretools/azureexplorer/editors/webapp/WebAppBasePropertyEditor.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -615,6 +615,7 @@ private void resetTblAppSettings(Map<String, String> map) {
615615
item.setText(new String[]{key, map.get(key)});
616616
}
617617
updateTableActionBtnStatus(true);
618+
updateSaveAndDiscardBtnStatus();
618619
}
619620

620621
private void readTblAppSettings() {

PluginsAndFeatures/azure-toolkit-for-eclipse/com.microsoft.azuretools.core/src/com/microsoft/azuretools/core/ui/SignInDialog.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,10 @@ protected Control createDialogArea(Composite parent) {
143143
rbtnDevice.addSelectionListener(new SelectionAdapter() {
144144
@Override
145145
public void widgetSelected(SelectionEvent e) {
146-
enableAutomatedAuthControls(false);
146+
if(rbtnDevice.getSelection()) {
147+
rbtnDevice.setFocus();
148+
enableAutomatedAuthControls(false);
149+
}
147150
}
148151
});
149152
rbtnDevice.setText("Device Login");
@@ -165,7 +168,12 @@ public void widgetSelected(SelectionEvent e) {
165168
rbtnAutomated.addSelectionListener(new SelectionAdapter() {
166169
@Override
167170
public void widgetSelected(SelectionEvent e) {
168-
enableAutomatedAuthControls(true);
171+
if (rbtnAutomated.getSelection()) {
172+
// Set the radio button to be focused or the default one will be selected when refresh
173+
// For issue https://github.com/microsoft/azure-tools-for-java/issues/3543
174+
rbtnAutomated.setFocus();
175+
enableAutomatedAuthControls(true);
176+
}
169177
}
170178
});
171179
rbtnAutomated.setText("Service Principal");

0 commit comments

Comments
 (0)