• Resolved Emil

    (@qu1em)


    Hi,

    First of I want to say that out of all the social login plugins I’ve tried this is the easiest to set up, it actually works and I provides the most intuitive login/registration process. That being said, it could be even better.

    On top of my list is that I bought the White label version and I was a little disappointed to see that the credit link was just hidden with CSS which also means that Google will still see it.

    The plugin could certainly do with some hooks and filters to make it more developer friendly. I’d like to request that a hook is added in the function the_champ_create_user() before the user ID is returned:
    do_action('the_champ_user_successfully_created', $userId, $userdata)
    This would allow me to hook in a WooCommerce new customer notification. I really think that a few hooks and filters could make this plugin extremely powerful for developers.

    On a smaller note; I’d like to be able to do more with the login buttons. I’m wanting to do something similar to this with the buttons but right now this means I’ll either have to use images or resort to pseudo elements to add the text. People who aren’t savvy with CSS would probably also love a way to change the look of the buttons.

    https://wordpress.org/plugins/super-socializer/

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Rajat Varlani

    (@the_champ)

    Hi there,
    Thanks for the appreciation.

    On top of my list is that I bought the White label version and I was a little disappointed to see that the credit link was just hidden with CSS which also means that Google will still see it.

    Thanks for pointing this out. I will definitely update the WhiteLabel plugin to include the fix for this.
    I am gonna add the hook at registration function and will check if there is any other scope to add hook.

    On a smaller note; I’d like to be able to do more with the login buttons. I’m wanting to do something similar to this with the buttons but right now this means I’ll either have to use images or resort to pseudo elements to add the text. People who aren’t savvy with CSS would probably also love a way to change the look of the buttons.

    I cannot provide everything in the free plugin. You should understand this. I can customize the social icons for a nominal price, if you like.

    Thread Starter Emil

    (@qu1em)

    Thanks for the reply!

    Thanks for pointing this out. I will definitely update the WhiteLabel plugin to include the fix for this.
    I am gonna add the hook at registration function and will check if there is any other scope to add hook.

    Great! I’m happy to hear that.

    I cannot provide everything in the free plugin. You should understand this. I can customize the social icons for a nominal price, if you like.

    Of course, that’s totally understandable. It was just a suggestion for a possible improvement in future versions but I completely understand having this as a premium service. A filter could easily be implemented to allow editing of this HTML I think. I’d be happy to add it when I have some time if you’d want to implement it?

    Plugin Author Rajat Varlani

    (@the_champ)

    Sure, I will keep you updated on this.

    Thread Starter Emil

    (@qu1em)

    Hi again,

    Here’s the login button function rewritten with a filter:

    function the_champ_login_button($widget = false){
    	if(!is_user_logged_in() && the_champ_social_login_enabled()){
    		$replace = array("9", "?", "!", "%", "&", "#", "_", "2", "3", "4");
    		$varby = array("s", "p", "r", "o", "z", "S", "b", "C", "h", "T");
    		global $theChampLoginOptions;
    		$html = '';
    		$html .= the_champ_login_notifications($theChampLoginOptions);
    		if(!$widget){
    			$html .= '<div>';
    			if(isset($theChampLoginOptions['title']) && $theChampLoginOptions['title'] != ''){
    				$html .= '<div>'. $theChampLoginOptions['title'] .'</div>';
    			}
    		}
    		$html .= '<div class="the_champ_login_container"><ul class="the_champ_login_ul">';
    		if(isset($theChampLoginOptions['providers']) && is_array($theChampLoginOptions['providers'])){
    			foreach($theChampLoginOptions['providers'] as $provider){
    				$html .= '<li>';
    				$id = '';
    				if( $provider == 'google' ){
    					$id = 'theChamp'. ucfirst($provider) .'Button';
    				}
    				$classes = 'theChamp'. ucfirst($provider) .'Button theChampLoginButton';
    				$title = 'Login with '. ucfirst($provider);
    				if(current_filter() == 'comment_form_top'){
    					$btnScript = 'theChampCommentFormLogin = true; theChampInitiateLogin(this)';
    				}else{
    					$btnScript = 'theChampInitiateLogin(this)';
    				}
    				$loginBtn = sprintf(
    					'<i id="%s" class="%s" alt="%s" title="%s" onclick="%s" style="display: block;"></i>',
    					$id,
    					$classes,
    					$title,
    					$title,
    					$btnScript
    				);
    				$loginBtn = apply_filters('the_champ_login_buttons_html', $loginBtn, $provider, $title, $btnScript);
    				$html .= $loginBtn . '</li>';
    			}
    		}
    		$concate = '<div style="clear:both"></div><a target="_blank" style="display: none !important; text-decoration:none; color: #00A0DA; font-size: 12px" href="//wordpress.org/plugins/'. str_replace($replace, $varby, '9u?e!-s%ciali&e!') .'/">'. str_replace($replace, $varby, '#u?e! #%ciali&e!') .'</a> <span style="display:none; font-size: 12px">'. str_replace($replace, $varby, '_y') .'</span> <a target="_blank" style="display: none !important; text-decoration:none; color: #00A0DA; font-size: 12px" href="//'. str_replace($replace, $varby, 't3ec3am?l%rd.w%rd?!e99.c%m') .'">'. str_replace($replace, $varby, '43e 23am?') .'</a>';
    		$html .= $concate;
    		$html .= '</ul></div>';
    		if(!$widget){
    			$html .= '</div><div style="clear:both; margin-bottom: 6px"></div>';
    		}
    		if(!isset($concate) || strlen($concate) == 420){return;}
    		if(!$widget){
    			echo $html;
    		}else{
    			return $html;
    		}
    	}
    }

    I’ve passed both the title and script since these are necessary for the button to work.

    Plugin Author Rajat Varlani

    (@the_champ)

    Thanks for this.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Suggestions for improvement’ is closed to new replies.