• Resolved Bonaldi

    (@spameur)


    Hi,

    I suggest you some improvements to optimize your plugin.

    Missing string in elementor/includes/widgets/image.php, line 98
    You can replace with __( 'Link to', 'elementor' )

    You can too translate system_schemes titles, in elementor/includes/schemes/color.php

    And elementor/includes/widgets-manager.php, line 135
    You can replace with __( 'No post_id', 'elementor' )

    Fontawesome current version can be updated to 4.6.3.

    Often, the last line of files is empty.

    Security question: do you really have to encode some code with base64 ?

    Hope it will help.
    Regards

    https://wordpress.org/plugins/elementor/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Yakir Sitbon

    (@kingyes)

    The Link to is for a11y only. It’s hide from the user, but I added the translate there too.

    And for the No post_id is not needed to get translate. It’s just for developer debug, so the end-user not need to see that message.

    Thanks 🙂

    Thread Starter Bonaldi

    (@spameur)

    OK, thanks.
    But you know, some newbies developers like have a maximum translated strings. For post_id ok, but no can be translated for a better understanding.
    Regards.

    Plugin Author ArielK

    (@arielk-1)

    It’s ok, we fix it for the next release

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Various ToDo's’ is closed to new replies.