Target _blank only option?

  • AWKampen

    (@awkampen)


    8 years, 5 months ago

    Hi,

    I was looking for a versatile Google Maps plugin for my site and I found yours. Great plugin and I really want to use it, but… I found a small error.

    I really want my locations to target _self and not _blank.
    It seems that you have the code for it in place already, but the classname of the form is wrong.

    In
    /wp-content/plugins/wp-google-map-plugin/modules/location/views/form.php
    Line 168, it now says:

    ‘class’ => ‘wpgmp_location_onclick_redirect wpgmp_location_onclick form-control’,

    This should be:

    ‘class’ => ‘wpgmp_location_onclick_custom_link wpgmp_location_onclick form-control’,

    Do you think it’s possible to change this in the next update? I would hate to have to do it manually all the time.

    Thanks again for a wonderful plugin!
    I’m looking forward to your response.

Viewing 4 replies - 1 through 4 (of 4 total)
  • erdivyang10

    (@erdivyang10)

    8 years, 5 months ago

    Hello,

    Thank you very much for your valuable suggestion, we will definitely think about it and integrate into our future version.

    Thank You

    Thread Starter AWKampen

    (@awkampen)

    8 years, 4 months ago

    Hello again,

    You have just released a new version of your wonderful plugin. Great!
    Unfortunately you didn’t incorporate my earlier suggestion. I had to replace the faulty class again by hand, to be able to see the ‘open new tab’ option again.
    You probably overlooked it. No worries!
    The code in question can now be found on line 168.
    Please incorporate this rectification, because it makes the application so much more powerful.

    Keep up the good work and thanks again for this fantastic plugin!

    • This reply was modified 8 years, 4 months ago by AWKampen.
    Thread Starter AWKampen

    (@awkampen)

    8 years, 3 months ago

    Hello again,

    Just installed another update for your wonderful plugin, but unfortunately the wrong piece of code has still not been rectified.
    It’s a bit of a hassle to keep changing this manually after every update.
    Please look into this. It’s a significant update to this fantastic plugin!

    Thread Starter AWKampen

    (@awkampen)

    8 years ago

    Hi there,

    Just a quick reminder, because once again I had to manually alter line 168 to get the target option visible.
    I do hope you will consider correcting this for the next version.
    Still loving this plugin though! 😀

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

The topic ‘Target _blank only option?’ is closed to new replies.

  • 4 replies
  • 2 participants
  • Last reply from: AWKampen
  • Last activity: 8 years ago
  • Status: not resolved