(Suggestion) Label Tool buttons

[Michael]

I found the $file_tools output as Images only confusing. I added words to the a href’s perhaps there could a custom checkbox to label or not to label the tools.

`$file_tools .= ‘<a href=”#” class=”nm-file-tools btn btn-primary u_i_c_tools_del” title=”‘.__(‘Remove’, ‘ppom’).'”>Remove <span class=”fa fa-times”></span></a>’;
$file_tools .= ‘<a href=”#” data-toggle=”modal” data-target=”#modalFile’.esc_attr($file_id).'” class=”btn btn-primary”>View <span class=”fa fa-expand”></span></a>’;

if(isset($settings[‘photo_editing’]) && $settings[‘photo_editing’] == ‘on’){

if( isset($settings[‘editing_tools’]) ) {
parse_str ( $settings[‘editing_tools’], $parsed_editing_tools );
$editing_tools = implode(“,”, $parsed_editing_tools[‘editing_tools’]);
}

$cropping_preset = isset($settings[‘aviary_crop_preset’]) ? $settings[‘aviary_crop_preset’] : ”;

$file_tools .= ‘<a href=”javascript:;” onclick=”launch_aviary_editor(\”.$file_id.’\’, \”.ppom_get_dir_url() . $file_name.’\’, \”.$file_name.’\’, \”.$editing_tools.’\’, \”.$cropping_preset.’\’)” class=”nm-file-tools btn btn-primary” title=”‘.__(‘Edit image’, ‘ppom’).'”>Edit <span class=”fa fa-pencil”></span></a>’;`

[nmedia]

Hi Michael,

Thanks for your suggestions, but I am not sure what changes you made? Can you please tell me changes so we can check and implement if it worth 😉

[Michael]

All I did was stick the word before the span of the button, nothing fancy no config control. IE

”>Remove <span

and

.’”>Edit <span

Obviously you could make that a variable or switch if you want to be able to turn it on/off

[nmedia]

Sorry, still can’t get it … Any screenshot?

[Michael]

Sure

-Mike

[nmedia]

Ah I see, you mean this. Well we will add this.
Thanks.

[Jane Brian]

HI
It has been a long while for this topic, so we are closing this.

[Author]

Posts