Skip to content

Conversation

@peppedeka
Copy link
Contributor

implementation of clear button via eventEmitter from dummy component to root component

@andrea82
Copy link
Member

Hi @peppedeka
thanks for your contribution. could you move the clear button inside the "REDIS RESPONSES" h3 tag and position it on extreme right ? Also if we could use an icon instead of the "clear" label would be great!
Thanks so much!

@peppedeka
Copy link
Contributor Author

HI @andrea82
fixed!

@andrea82
Copy link
Member

Ok... almost perfect. Is better to keep layout style for clear button as well, so if you could add the following style for icon and remove background and border of button style will be perfect.
.h7 { font-size: 1rem; color: #7a0c00; font-weight: bolder; }

thanks.

@peppedeka peppedeka force-pushed the add-crear-button-output-panel branch from ce7a011 to 5b67aac Compare December 14, 2019 10:40
@peppedeka
Copy link
Contributor Author

done!

@andrea82 andrea82 merged commit 6a5298d into acadevmy:develop Dec 14, 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.

2 participants