Skip to content

Commit 8e2e25e

Browse files
author
martgil
committed
pr reviews: use .is(':visible')
1 parent 5fdca47 commit 8e2e25e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ export class GmailElementReplacer extends WebmailElementReplacer {
283283
};
284284

285285
private addMenuButton = (replyOption: ReplyOption, gmailContextMenuBtn: string) => {
286-
if ($(gmailContextMenuBtn).css('display') === 'block') {
286+
if ($(gmailContextMenuBtn).is(':visible')) {
287287
const button = $(this.factory.btnSecureMenuBtn(replyOption)).insertAfter(gmailContextMenuBtn); // xss-safe-factory
288288
button.on(
289289
'click',
@@ -924,8 +924,10 @@ export class GmailElementReplacer extends WebmailElementReplacer {
924924
this.addMenuButton('a_reply', '#r');
925925
this.addMenuButton('a_reply_all', '#r2');
926926
this.addMenuButton('a_forward', '#r3');
927+
} else {
928+
Catch.setHandledTimeout(addSecureOptionsToGmail, 100);
927929
}
928930
};
929-
Catch.setHandledTimeout(addSecureOptionsToGmail, 100);
931+
addSecureOptionsToGmail();
930932
};
931933
}

0 commit comments

Comments
 (0)