Multiple suggestions

  • Resolved tmuikku

    (@tmuikku)


    9 years, 5 months ago

    Hi,

    I’ve downloaded the free version and the WP Advanced PDF plugin version 1.1.2 and it seems to work just fine.
    I have some suggestions for improvement though:

    1. Please make all text translatable, now there is many words that are not in gettext functions
    2. Please utilize nonces
    3. Please add filters for header and footer content so one can add dynamic content such as a date or a permalink
    4. Please add filter or utilize WordPress template system for the “send to email” popup form
    5. Please remove inline styling, use a CSS file for styles, which you have queued already
    6. Please add option to allow direct download for the not logged in users as well
    7. Filter for default settings would be nice

    In general, bring the source to WordPress standards and general guidelines, fix typos etc. to improve quality.

    • This topic was modified 9 years, 5 months ago by tmuikku.
Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author cedcommerce

    (@cedcommerce)

    9 years, 5 months ago

    Hello tmuikku,

    Thanks a lot for your suggestions we really appreciate it and we noted down your each and every suggestion, and we will defenetly add important features and fix issues you have mentioned in suggestions,

    Apart from this we must inform you that in premium version of this plugin we had added too many features including your suggestion no.- 6 so please go through that also.

    Your Suggestions are most valuable for us.

    Thanks
    CedCommerce

    Thread Starter tmuikku

    (@tmuikku)

    9 years, 5 months ago

    Cool!
    There was another important one I forgot to add. The email popup HTML is echoed to html <head> section with wp_head action hook. This is bad, please change the action hook to wp_footer.
    Thanks.

    Plugin Author cedcommerce

    (@cedcommerce)

    9 years, 5 months ago

    Hello tmuikku,

    Ok, we had noted down this point too.

    Thanks
    CedCommerce

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Multiple suggestions’ is closed to new replies.

Tags