wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1110155 - Fixed renamed Loop room not reflected into the panel. r=Standard8

Browse Source
This commit is contained in:
Nicolas Perriault 2014-12-23 11:54:32 +01:00
parent 22c59567cc
commit c98b60738c
3 changed files with 18 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
browser/components/loop/content/js/panel.js
View File

@ -568,7 +568,14 @@ loop.panel = (function(_, mozL10n) {
handleFormSubmit: function(event) {
event.preventDefault();
this.props.onChange(this.state.text);
// While we already validate for a non-empty string in the store, we need
// to check it at the component level to avoid desynchronized rendering
// issues.
if (this.state.text.trim()) {
this.props.onChange(this.state.text);
} else {
this.setState({text: this.props.text});
}
this.setState({edit: false});
},

9
browser/components/loop/content/js/panel.jsx
View File

@ -568,7 +568,14 @@ loop.panel = (function(_, mozL10n) {
handleFormSubmit: function(event) {
event.preventDefault();
this.props.onChange(this.state.text);
// While we already validate for a non-empty string in the store, we need
// to check it at the component level to avoid desynchronized rendering
// issues.
if (this.state.text.trim()) {
this.props.onChange(this.state.text);
} else {
this.setState({text: this.props.text});
}
this.setState({edit: false});
},

3
browser/components/loop/content/shared/js/roomStore.js
View File

@ -423,10 +423,11 @@ loop.store = loop.store || {};
* @param {sharedActions.RenameRoom} actionData
*/
renameRoom: function(actionData) {
var oldRoomName = this.getStoreState("roomName");
var newRoomName = actionData.newRoomName.trim();
// Skip update if name is unchanged or empty.
if (!newRoomName || this.getStoreState("roomName") === newRoomName) {
if (!newRoomName || oldRoomName === newRoomName) {
return;
}