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

UI Enhancement; updating plugin title. See: websharks/comment-mail#131 #133

Closed
wants to merge 2 commits into from

Conversation

kristineds
Copy link
Contributor

Closes #131

@raamdev
Copy link
Contributor

raamdev commented Jul 29, 2015

Reviewed. Looks good to me!

@jaswrks
Copy link

jaswrks commented Jul 29, 2015

For the Lite version, can you check and see what it looks like with Lite instead of Pro :-) If it's too jumbled we can just exclude this from the Lite version.

@kristineds
Copy link
Contributor Author

can you check and see what it looks like with Lite instead of Pro

Ha! Sorry. Was so used to submitting PRs for both Pro and Lite. That has been changed. :)

@jaswrks
Copy link

jaswrks commented Jul 29, 2015

haha.. No worries. Can you post a screenshot of what this looks like for me?

@raamdev
Copy link
Contributor

raamdev commented Jul 29, 2015

For the Lite version,

Ha, I totally missed that too.

@kristineds
Copy link
Contributor Author

screen shot 2015-07-30 at 2 49 30 am

@raamdev
Copy link
Contributor

raamdev commented Jul 29, 2015

we can just exclude this from the Lite version.

I would just exclude the superscript from the Lite version altogether--adding Lite is too jumbled IMO.

@jaswrks
Copy link

jaswrks commented Jul 29, 2015

Thank you :-) Yep, that looks a bit hard to read with the extra letter. Thoughts on this? I say we could just exclude this from the Lite version. It works great for the Pro copy though.

@jaswrks
Copy link

jaswrks commented Jul 29, 2015

I would just exclude the superscript from the Lite version altogether--adding Lite is too jumbled IMO.

Copy that.

@jaswrks jaswrks closed this Jul 29, 2015
@jaswrks jaswrks deleted the feature/131 branch August 4, 2015 17:11
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.

None yet

3 participants