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

RC :: Enhancement :: Pro Preview Link #227

Closed
jaswrks opened this Issue Feb 11, 2016 · 10 comments

Comments

Projects
None yet
3 participants
@jaswrks
Member

jaswrks commented Feb 11, 2016

The RC is looking really great! Fantastic work, great improvements!


To take the recent navigation bar improvements to the next step, in the Lite version, I suggest moving the Pro Preview link into the main navigation area. Or, if that's not desirable try to find a way to make this easier to see. In this particular plugin (because it's a set-it-and forget-it kind of plugin), that Pro Preview link seems important. Ideally, it would be impossible to install the free version and not see there are pro features as soon as you begin to explore what it has to offer; i.e., one of the first things that you see.

2016-02-11_11-20-32

@raamdev raamdev added the todo label Feb 11, 2016

@raamdev raamdev modified the milestones: Next Release, Future Release Feb 11, 2016

@raamdev

This comment has been minimized.

Contributor

raamdev commented Mar 16, 2016

I'm not sure when it happened, but it looks like this was done at some point recently:

2016-03-16_12-11-15

However my feeling is that these two links should go after "Email Templates" and "Site Templates":

2016-03-16_12-13-09

Also, while working on this we need to fix an issue with those menu links colliding with the bar below them when the browser is resized:

2016-03-16_12-14-05

@raamdev raamdev added bug user experience and removed todo labels Mar 16, 2016

@raamdev

This comment has been minimized.

Contributor

raamdev commented Mar 16, 2016

Related: #236

@renzms

This comment has been minimized.

Contributor

renzms commented Mar 19, 2016

@raamdev

From #236, I think that solves the smaller browser issue of the links overlapping.

I tested with a smaller browser size and with the notifications on top. The links stay in place, please see screen shot:
screen shot 2016-03-20 at 2 03 23 am

Here is my test with it at regular browser size:

screen shot 2016-03-20 at 2 03 45 am

Is that okay, or should the Preview Pro Features and Pro Upgrade be below the template links? It seems crowded when it extends all the way to the right with all the links on one line.

@raamdev

This comment has been minimized.

Contributor

raamdev commented Mar 19, 2016

@renzms I just built and tested Comment Mail Lite from the latest dev branch, which includes the work from #236, and I'm still seeing an issue (Google Chrome on OS X):

2016-03-19_13-17-25

@renzms

This comment has been minimized.

Contributor

renzms commented Mar 22, 2016

@raamdev

Hmm, I'm also using Google Chrome on OS X:

Mac  System Version:    OS X 10.11.3 (15D21)
Chrome Version 49.0.2623.87 (64-bit)

tested Comment Mail Lite

In your screenshot it says Comment Mail Pro, or is that just because of the Phing Build version?
Because in the Lite Version you should not be able to remove the logo as that's locked under a Pro Feature under Misc. UI Settings. Even when previewing that Pro Feature, it should not save as it is grayed out.

Were there any recent merges after you tested and reviewed #236, that may have overwritten something in the CSS?

@raamdev

This comment has been minimized.

Contributor

raamdev commented Mar 23, 2016

In your screenshot it says Comment Mail Pro, or is that just because of the Phing Build version?

The menu item says 'Pro' because we haven't finished updating the unified codebase to strip out all Pro-related things during a build (see #237).

So I was testing the Lite build (.~build/comment-mail/.~build/comment-mail.zip after running phing). Is that what you were testing?

@renzms

This comment has been minimized.

Contributor

renzms commented Mar 23, 2016

So I was testing the Lite build (.~build/comment-mail/.~build/comment-mail.zip after running phing). Is that what you were testing?

Nope, I will test with that version then. One question though, if I'm testing the Lite version of it, should I remove the logo? As that's a Pro feature, not sure ifs automatically disabled since the Pro-related features are still in the build.

@raamdev

This comment has been minimized.

Contributor

raamdev commented Mar 23, 2016

The logo doesn't show up in the Lite version.

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

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

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

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

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

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

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

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

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

@raamdev

This comment has been minimized.

Contributor

raamdev commented May 17, 2016

Next Release Changelog:

  • Enhancement: Minor improvements to the Options Page menu links and positioning of the Pro Preview link. Props @renzms. See Issue #227.

@raamdev raamdev closed this May 17, 2016

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

Phing release of v160618 with the following changes:
- **Restructured Codebase**: The codebase has been completely restructured to improve performance, enhance flexibility, and make it easier to build in new features! Props @jaswsinc. See [Issue #150](#150).
- **Comment Mail Pro Upgrade Notice: Incompatible Advanced Templates.** This version of Comment Mail includes a rewritten and improved codebase. This rewrite, however, came with the unfortunate side effect of breaking backwards compatibility with Advanced Templates that were customized in a previous version of Comment Mail Pro.

     If you are currently using Comment Mail Pro and you've customized your Advanced Templates, all of your customized Advanced Templates will be backed up and the templates will then be reset to their new defaults. You will find the backup of your old customized template appended to the bottom of the new template, separated with a  <code>Legacy Template Backup</code> PHP comment. See [example screenshots](#238 (comment)).

     Note: This change has no effect on Simple templates—only Advanced Templates are affected. Advanced Templates are a Pro-only feature, so this notice only applies to Comment Mail Pro. See [Issue #238](#238).
- **Bug Fix**: Fixed a bug where `esc_html()` was being used where `esc_sql()` should've been used. Props @jaswsinc @kristineds. See [Issue #268](#268).
- **Bug Fix**: Fixed a bug that in some scenarios resulted in a "DB table creation failure" error when activating the plugin. Props @thienhaxanh2405, @PanNovak, @kristineds, and @jaswsinc. See [Issue #260](#260).
- **Bug Fix**: Fixed a bug where "New reply" notification emails were not being parsed properly by some Hotmail accounts and were showing up as blank. Props @kristineds. See [Issue #259](#259).
- **Bug Fix**: Fixed a bug that allowed spam comments to create subscriptions in Comment Mail when using Akismet. Props @IvanRF. See [Issue #250](#250).
- **Bug Fix** (Pro): When Chrome or Firefox Autofill Username/Password was enabled, the Comment Mail Pro Updater fields would incorrectly be autofilled by the browser with invalid credentials. This has been fixed. Props @renzms. [Issue #274](#274).
- **Bug Fix**: Fixed a bug where the cron job for the Queue Processor could get deleted and never recreated, which would result in notifications getting stuck in the Mail Queue and not being sent out. If you ever installed Comment Mail and then deleted it (without first disabling Data Safeguards), and then installed Comment Mail again, you were probably affected by this issue. This release fixes the issue and makes the cron setup more robust. Props @kristineds, @renzms, @jaswsinc, and @IvanRF for help testing. See [Issue #194](#194) and [Issue #173](#173).
- **Bug Fix:** Fixed a bug where a subscriber who selected Hourly Digest and who had never been notified before could, in some scenarios, have their subscription treated instead as a Weekly Digest. This bug was found and fixed during the codebase restructuring. Props @jaswsinc. See [Issue #150](#150) and additional discussion in [Issue #173](#173 (comment)).
- **Bug Fix:** Fixed a bug where in some scenarios Mail Queue entries for Digest Notifications that should have been held for sending later were not being held and were also not being sent. They also would not have shown up in the Mail Queue Event Log. This bug was found and fixed during the codebase restructuring. Props @jaswsinc. See [Issue #150](#150) and additional discussion in [Issue #173](#173 (comment)).
- **Enhancement**: Minor improvements to the Options Page menu links and positioning of the Pro Preview link. Props @renzms. See [Issue #227](#227).
- **Enhancement**: It's now possible to use the following shortcodes in the Email Footer Tag for Email Footer Templates: `[home_url]`, `[blog_name_clip]`, and `[current_host_path]`. Props @kristineds and @IvanRF. See [Issue #246](#246).
- **Enhancement**: Improved the Subscriptions meta box that appears on the Post Edit screen. For each subscription, the meta box now lists the full name and email address, the date the subscription was created, and a view link that allows you to view/edit the subscription. Props @kristineds. See [Issue #231](#231).
- **UX Enhancement (Pro)**: Improved the Dashboard notice that appears when you try to enable the Pro version of Comment Mail when the Lite version is currently enabled. Props @kristineds @jaswsinc. See [Issue #230](#230).
- **UX Enhancement**: When Subscribing Without Commenting, the Add New Subscription form now pre-populates the Name and Email address fields whenever possible. Props @kristineds. See [Issue #204](#204).
- **UI Enhancement**: Dashboard notices generated by Comment Mail now use the WordPress-style dismiss button to keep things consistent. Props @kristineds. See [Issue #193](#193).
@raamdev

This comment has been minimized.

Contributor

raamdev commented Jun 19, 2016

Comment Mail v160618 has been released and includes changes from this GitHub Issue. See the v160618 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 (#227).

@websharks websharks locked and limited conversation to collaborators Jun 19, 2016

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