Skip to content

Commit 4af70f1

Browse files
committed
pr reviews: make code concise
1 parent 18564f0 commit 4af70f1

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

extension/js/content_scripts/webmail/gmail/gmail-element-replacer.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -281,19 +281,19 @@ export class GmailElementReplacer extends WebmailElementReplacer {
281281
return !!$('iframe.pgp_block').filter(':visible').length;
282282
};
283283

284+
private addMenuButton = (action: 'reply' | 'forward', selector: string) => {
285+
const gmailActionsMenuContainer = $(this.sel.msgActionsMenu).find(selector);
286+
const button = $(this.factory.actionsMenuBtn(action)).insertAfter(gmailActionsMenuContainer); // xss-safe-factory
287+
button.on(
288+
'click',
289+
Ui.event.handle((el, ev: JQuery.Event) => this.actionActivateSecureReplyHandler(el, ev))
290+
);
291+
};
292+
284293
private replaceActionsMenu = () => {
285-
const gmailActionsMenuContainer = $(this.sel.msgActionsMenu);
286294
if ($('.action_menu_message_button').length <= 0) {
287-
const menuSecureReplyBtn = $(this.factory.actionsMenuBtn('reply')).insertAfter($(gmailActionsMenuContainer).find('#r')); // xss-safe-factory
288-
const menuSecureForwardBtn = $(this.factory.actionsMenuBtn('forward')).insertAfter($(gmailActionsMenuContainer).find('#r3')); // xss-safe-factory
289-
menuSecureReplyBtn.on(
290-
'click',
291-
Ui.event.handle((el, ev: JQuery.Event) => this.actionActivateSecureReplyHandler(el, ev))
292-
);
293-
menuSecureForwardBtn.on(
294-
'click',
295-
Ui.event.handle((el, ev: JQuery.Event) => this.actionActivateSecureReplyHandler(el, ev))
296-
);
295+
this.addMenuButton('reply', '#r');
296+
this.addMenuButton('forward', '#r3');
297297
}
298298
};
299299

0 commit comments

Comments
 (0)