(In reply to Magnus Melin [:mkmelin] from comment #11) > Comment on attachment 9072528 [details] [diff] [review] > Bug-1523419_chat-room-edit-topic.patch > > Review of attachment 9072528 [details] [diff] [review]: > ----------------------------------------------------------------- > > Looks good. I would prefer to use better variable names. topic instead of > elt, and topicInput instead of eltInput Cool, I will update the patch ASAP.
Bug 1523419 Comment 12 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Magnus Melin [:mkmelin] from comment #11) > > Looks good. I would prefer to use better variable names. topic instead of > elt, and topicInput instead of eltInput Cool, I will update the patch ASAP.