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

Improve Pro Options Panel header when logo is hidden #284

Closed
raamdev opened this Issue Jun 18, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@raamdev
Contributor

raamdev commented Jun 18, 2016

When the Comment Mail logo is hidden (which true by default in the Pro version), the header of the options pages contains a lot of unused whitespace.

2016-06-18_15-47-38

My feeling is we can probably move or get rid of the "Restore" button—that button is something that will so rarely be used that I don't think it needs to be so prominent. Maybe it could be moved to the links menu instead:

2016-06-18_15-57-36

@raamdev raamdev added this to the Future Release milestone Jun 18, 2016

@raamdev raamdev added user interface and removed enhancement labels Jun 19, 2016

renzms added a commit to websharks/comment-mail-pro that referenced this issue Jun 24, 2016

@raamdev raamdev modified the milestones: Next Release, Future Release Jul 12, 2016

renzms added a commit to websharks/comment-mail-pro that referenced this issue Jul 12, 2016

raamdev added a commit to websharks/comment-mail-pro that referenced this issue Jul 29, 2016

@raamdev

This comment has been minimized.

Contributor

raamdev commented Jul 29, 2016

Next Release Changelog:

  • UI Enhancement: Improved the nav bar at the top of the options pages to reduce unnecessary whitespace. Also moved the Restore button to the nav bar so that it's not so prominent. Props @renzms. See Issue #284.

@raamdev raamdev closed this Jul 29, 2016

@raamdev raamdev modified the milestones: Next Release, Future Release Jul 29, 2016

raamdev added a commit that referenced this issue Aug 18, 2016

Phing release of v160818 with the following changes:
- **Bug Fix**: Fixed a bug where the "My Comment Subscriptions" link would appear on the Add New Subscription page (when Subscribing without Commenting) and would lead to a page that displayed an error message stating that there were no subscriptions to list. That link is now hidden when there are no subscriptions to list. Props @Reedyseth @kristineds. See [Issue #229](#229).
- **Bug Fix** (Pro): Removed an erroneous anchor tag in the Advanced Template for Comment Notification Message Body. Props @kristineds. See [Issue #287](#287).
- **UI Enhancement:** Improved the nav bar at the top of the options pages to reduce unnecessary whitespace. Also moved the Restore button to the nav bar so that it's not so prominent. Props @renzms. See [Issue #284](#284).
- **UI Enhancement:** Added links to the Comment Mail [Twitter](http://twitter.com/CommentMail) and [Facebook](https://www.facebook.com/Comment-Mail-565683256946855/) pages to the nav bar on the options page. Props @renzms. See [Issue #286](#286).
- **UX Enhancement:** Removed IP address information from email notification templates to better comply with data protection laws in certain countries. Props @kristineds. See [Issue #288](#288).
- **SEO Improvement:** Added `rel="nofollow"` to the "Subscribe without Commenting" link and "Manage Subscriptions" link on the comment subscription form to avoid indexing or transferring PageRank. Props @IvanRF. See [Issue #80](websharks/comment-mail-pro#80).
- Removed several development-only files from the distributable that were inadvertently included during the build process. See [Issue #285](#285).
- Added Renz Sevilla (`renzms`) to the contributors list.
@raamdev

This comment has been minimized.

Contributor

raamdev commented Aug 18, 2016

Comment Mail v160818 has been released and includes changes from this GitHub Issue. See the v160818 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#284).

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