• Resolved harvoolio

    (@harvoolio)


    John,

    I wanted to move the icon in the toggle to the end instead of the beginning. So, I can change one line in shortcodes.php

    from

    $r = ‘<div class=”arconix-toggle-wrap’. $css . ‘”>’ . ‘<div class=”arconix-toggle-title ‘ . $load . ‘”>’ . $icon . $title . ‘</div>’;

    to

    $r = ‘<div class=”arconix-toggle-wrap’. $css . ‘”>’ . ‘<div class=”arconix-toggle-title ‘ . $load . ‘”>’ . $title . $icon . ‘</div>’;

    But is there a way to do this without changing plugin code? Is there a PhP function I should be writing?

    Thanks.

    https://www.ads-software.com/plugins/arconix-shortcodes/

Viewing 15 replies - 1 through 15 (of 20 total)
  • Plugin Author John Gardner

    (@jgardner03)

    Hi,

    Thanks for using my plugin. Normally I’m really good about offering filter options which would allow you to make changes to plugin output without altering plugin code, but in this case my shortcodes fall short. Right now the only way to do what you want is to alter plugin code. I’ll be changing that in a future release.

    Thanks

    Thread Starter harvoolio

    (@harvoolio)

    Thanks.

    I have been trying to learn PhP. I was hoping there was something like this psuedo-code (not close at all but you get the idea) I could put in my Functions area and leave the code alone:

    add_filter( ‘POINT TO APPROPRIATE FUNCTION WITHIN ARCONIX’ , ‘order_toggle’ );

    function order_toggle($order) {

    $r = ‘<div class=”arconix-toggle-wrap’. $css . ‘”>’ . ‘<div class=”arconix-toggle-title ‘ . $load . ‘”>’ . $title . $icon . ‘</div>’;

    return $order;

    }

    Plugin Author John Gardner

    (@jgardner03)

    Your approach would absolutely be the correct one (though technically you’d return $r and not $order, but that’s minor) if I had set up my code to allow that to happen. Make the change you need in the core files. I’ll update this thread when I’ve made the proper enhancement in the plugin.

    Thread Starter harvoolio

    (@harvoolio)

    Cool. Thanks. Replacing the plugin is fine – I am just trying to learn. Thank you for the quick response.

    Thread Starter harvoolio

    (@harvoolio)

    I could create another tag, but one quick question. Is there a way to make the toggle title tags header tags (I think in an early release you had changed this)?

    Thanks.

    Plugin Author John Gardner

    (@jgardner03)

    Yeah, they used to be header tags but there was some kind of issue with them, which is why I switched to a div. All you’d have to do is replace

    $r = '<div

    with

    $r = '<h3

    and replace the closing div w/ the appropriate closing header.

    Thread Starter harvoolio

    (@harvoolio)

    Thanks. What about making the text linkable? I tried putting in tags but it did not like it.

    So I then decided not to have a title and use normal tags (outside of the toggle). But I couldn’t figure out the CSS to make the toggle title inline (remember it is now after the tag) to the tag I had (the problem is the div.arconix-toggle-wrap).

    If there was no arconix-toggle-wrap I could just write:

    .list-toggle div.arconix-toggle-title {display: inline;}

    HTML

    Website
    [ac-toggle css=”list-toggle”]
    INSERT TOGGLE TEXT
    [/ac-toggle]

    OUTPUT

    Website
    <div class=”arconix-toggle-wrap list-toggle”>
    <div class=”arconix-toggle-title toggle-closed”>
    <i class=”fa fa-plus-square”></i>
    </div>
    <div class=”arconix-toggle-content” style=”display: none;”>
    INSERT TOGGLE TEXT
    </div>
    </div>

    CSS (If there was no div.arconix-toggle-wrap.)
    .list-toggle div.arconix-toggle-title {display: inline;}

    Thanks.

    Thread Starter harvoolio

    (@harvoolio)

    Oops WordPress converted my Website code. Website is in a tags.

    Plugin Author John Gardner

    (@jgardner03)

    Sorry, I’m a bit confused about the issue you’re having. The icon is already set to display inline. Are you talking about aligning it with the right side of the toggle?

    Thread Starter harvoolio

    (@harvoolio)

    I am trying to make the text within toggles linkable (a tags).

    I tried two solutions:
    (1) Make the Toggle Title Text into a tags. (did not work).
    (2) Do not use the Toggle Title Text. Instead put a blank Toggle Title Text and just put an a tag (so I would click just the plus to open and close the toggle). To do this I would need to make the a tag I entered in HTML inline with the blank Toggle Title (which contains the i tag and toggle). However the toggle has a wrap so I could not get the CSS to work.

    Did I explain this well?

    Plugin Author John Gardner

    (@jgardner03)

    You can make text within toggle content areas linkable, but there’s no way to make hyperlink the toggle titles.

    1. Anchor tags won’t work in the toggle title under any condition, as you discovered, because clicking on the toggle title is supposed to reveal the hidden content panel so browsers ignore any anchor entered. You can put hyperlinks on text in the content area (the space between the [toggle][/toggle]).
    2. You can enter a blank toggle title and the toggle will still open and close as expected. You can see that in action here: https://demo.arconixpc.com/box-test/

    That help at all?

    Thread Starter harvoolio

    (@harvoolio)

    Thanks.

    Assume that you had the blank toggle in the example. Also, assume that you got rid of the box around the toggle. Could you put normal text on the same line as the +/-. So this text would serve as a de facto title. You could then make this normal text whatever tag you wanted including tags.

    Thanks. I am going to put sample code below and I am going to wrap my a tags in parenthesis instead of <> so they do not turn into links.

    CODE IN WORDPRESS

    (a href=”https://website.com&#8221;)Website(/a)
    [ac-toggle css=”list-toggle”]
    INSERT TOGGLE TEXT
    [/ac-toggle]

    OUTPUT IN HTML
    (a class=”a_tag” href=”https://website.com&#8221;)Website(/a)
    <div class=”arconix-toggle-wrap list-toggle”>
    <div class=”arconix-toggle-title toggle-closed”>
    <i class=”fa fa-plus-square”></i>
    </div>
    <div class=”arconix-toggle-content” style=”display: none;”>
    INSERT TOGGLE TEXT
    </div>
    </div>

    All I want to do is make my a.a_tag and di.arconix-toggle-title inline. If there was no wrap, all I would need to do is write one line of CSS.

    .list-toggle div.arconix-toggle-title {display: inline;}

    Did I explain this well? Thanks.

    Plugin Author John Gardner

    (@jgardner03)

    can you put backticks around your code? html not in between those ticks gets parsed and it adds hyperlinks to things which makes it hard to understand. Backtick, btw is to the left of the 1 key on your keyboard (beneath the tilde)

    Thread Starter harvoolio

    (@harvoolio)

    Look at next post.

    Thread Starter harvoolio

    (@harvoolio)

    Thanks for teaching me something new. I can now use the proper a tags.

    If I put the following code in WordPress

    <a href="https://website.com">Website</a>
    [ac-toggle css="list-toggle"]
    INSERT TOGGLE TEXT
    [/ac-toggle]

    it produces the following HTML

    <a href="https://website.com">Website</a>
    <div class="arconix-toggle-wrap list-toggle">
      <div class="arconix-toggle-title toggle-closed">
        <i class="fa fa-plus-square"></i>
      </div>
      <div class="arconix-toggle-content" style="display: none;">
      INSERT TOGGLE TEXT
      </div>
    </div>

    If the following were not there

    <div class="arconix-toggle-wrap list-toggle">
    ...
    </div>

    I could just write the following CSS to make everything work.

    .list-toggle div.arconix-toggle-title {display: inline;}

    Thanks.

Viewing 15 replies - 1 through 15 (of 20 total)
  • The topic ‘Change icon order in toggle without changing code’ is closed to new replies.