Skip to content
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

add the debug plugin #2232

Merged
merged 5 commits into from
Aug 7, 2019
Merged

add the debug plugin #2232

merged 5 commits into from
Aug 7, 2019

Conversation

pldespaigne
Copy link
Contributor

Hello ! I made this PR to add my debug plugin, it will help plugin devs debugging their plugins, and remix devs testing the plugin-api !
Also sources are available here if you want to have a look.

Hello ! I made this PR to add my debug plugin, it will help plugin devs debugging their plugins, and remix devs testing the plugin-api !
Also sources are available [here](https://github.com/pldespaigne/remix-debug-plugin) if you want to have a look.
version: '0.1.0-alpha',
url: 'https://remix-debug-a.surge.sh',
icon: 'https://remix-debug-a.surge.sh/icon.png',
location: 'sidePanel'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wouldn't it be better in the mainpanel ? As most of the plugins are inside the sidepanel, you can have both of them on the same page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That make sense ! I will test the ui in the main panel then

events: [],
version: '0.1.0-alpha',
url: 'https://remix-debug-a.surge.sh',
icon: 'https://remix-debug-a.surge.sh/icon.png',
Copy link
Collaborator

Choose a reason for hiding this comment

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

we use a very similar icon for our native Debug plugin. both will be shown on Vertical panel when activated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah I wasn't sure about my icon, I try to mix the debug icon with the plugin manager icon, I can change that without changing the icon's url. Also I'm open to any icon suggestion

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://fontawesome.com/icons/spider?style=solid ? :) I'm not sure either, but it should be different

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll do it this evening

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi ! I have changed the icon, what do you think now ?
debug plugin icon

Copy link
Collaborator

Choose a reason for hiding this comment

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

I like it :))

Copy link
Collaborator

@LianaHus LianaHus left a comment

Choose a reason for hiding this comment

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

please try to find a different icon

@LianaHus
Copy link
Collaborator

LianaHus commented Aug 6, 2019

@pldespaigne could you please resolve the conflict so we can merge?

@yann300 yann300 merged commit 8d3a09f into ethereum:master Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants