Skip to content

Meta: Validate token in options page#3774

Merged
fregante merged 35 commits intomasterfrom
validate-token
Jan 6, 2021
Merged

Meta: Validate token in options page#3774
fregante merged 35 commits intomasterfrom
validate-token

Conversation

@yakov116
Copy link
Member

@yakov116 yakov116 commented Nov 27, 2020

  1. LINKED ISSUES:
    Closes Verify token in options #3752

  2. TEST URLS:
    The options Page

  3. SCREENSHOT:
    image

I also fixed the dropdown listener since it was broken

@yakov116
Copy link
Member Author

Hold off on the review I want to make changes

@yakov116 yakov116 removed the request for review from fregante November 27, 2020 16:53
@yakov116 yakov116 requested a review from fregante November 28, 2020 23:26
Comment on lines 160 to 162
setTimeout(() => {
void validateToken();
}, 100);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
setTimeout(() => {
void validateToken();
}, 100);
setTimeout(validateToken, 100);

But better yet if you dispatch an event at the end of https://github.com/fregante/webext-options-sync-per-domain/blob/dcc61fe58ab67d0982b2948cf3c1264375ef4096/index.ts#L129

Example: https://github.com/fregante/webext-options-sync/blob/master/index.ts#L253

… and then listen to that event specifically, instead of changing this listener

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to figure this out. But I could not get it to work by listening to the event.

@yakov116 yakov116 marked this pull request as ready for review January 1, 2021 15:14
yakov116 and others added 6 commits January 2, 2021 20:51
yakov116 and others added 2 commits January 3, 2021 17:34
Co-authored-by: Federico <me@fregante.com>
@yakov116
Copy link
Member Author

yakov116 commented Jan 4, 2021

Should all be good now. (I was looking over the code and I was wondering where my updated code was. Turns out I forgot to push :hehe)

@fregante fregante added meta Related to Refined GitHub itself and removed enhancement labels Jan 4, 2021
@fregante fregante changed the title Validate Token In Options Meta: Validate token in options page Jan 4, 2021
@fregante
Copy link
Member

fregante commented Jan 4, 2021

Full demo in Chrome (field size changed only in demo):

Dark Chrome:

Screen Shot

Screen Shot 1

Dark Firefox:

Screen Shot 2

Dark Safari:

Screen Shot 4

@yakov116
Copy link
Member Author

yakov116 commented Jan 5, 2021

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Related to Refined GitHub itself

Development

Successfully merging this pull request may close these issues.

Verify token in options

3 participants