-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: popup interface that does deconstructPayload and/or verifyOthersProve #107
Comments
This ext shall work on both chrome and firefox, desktop and mobile... It seems not work quite well on mobile, e.g., a new tab have to be opened... And is it supported by chrome too? Emm...The key issue is that, should we really need to expose the errors to end users, and which kinds of errors need we do?... a lot to consider before adopting that... |
mobile screen is narrow, so i don't really think turning a popup into a new tab is a major usability issue; in fact it probably is the only logical way to translate such a thing into mobile. and about that... jw wants to put it into the main option page as a dev option thing. i am currently working on that as it does not require me to write a new page lmao |
So... What do we need to put on the popup? e.g.:
|
Personally I think 2/3/4 is great. Don’t need to worry about 1 since we’re
not Adblock and won’t broke UIs. 5 is unnecessary for most user.
…On Thu, Aug 15, 2019 at 13:51 Sun Yi-Ran ***@***.***> wrote:
So... What do we need to put on the popup?
e.g.:
- A toggle to switch Maskbook on and off
- A quick preview to get / set current identity, persona, profile, etc.
- A inspector / aduitor to overview the current status, such as,
whether it is initialized, errors encountered, etc.
- A quick link to our settings page / dashboard
- A bottom navigator with several buttons to maskbook.com / github /
etc.
@neruthes <https://github.com/neruthes> @Artoria2e5
<https://github.com/Artoria2e5>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#107?email_source=notifications&email_token=ABTAVTPXWLBQQD7TVCNPKCDQETVEJA5CNFSM4ILSKVRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4K47UI#issuecomment-521523153>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABTAVTLVM3MY23LNNFG24WLQETVEJANCNFSM4ILSKVRA>
.
|
We don't have anything on persona yet, so 2 has to wait. 4 can be done with the // plz catch
<Button variant="contained" onClick={(e) => }>
OPTIONSSS!!!!!
</Button> |
@Artoria2e5 you can do it. Let's see @neruthes 's opinion |
Let the current path continue. We will clear things up in #70. |
For the latest Chrome Store ver, at least we have the pop up. : ) What's the next step and how about #70 ? |
This could be an easier path to working around some failures than going into the console and messing with the exposed services even for Power Users (TM).
WebExt popup
The text was updated successfully, but these errors were encountered: