e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
321 stars 214 forks source link

new forum post menu - not existing pref #4909

Open Jimmi08 opened 1 year ago

Jimmi08 commented 1 year ago

Bug Description

This is shortcode for content of

    function sc_post_content($parm = null)
    {

        $tp = e107::getParser();
        $pref = e107::getPref();
        $post = strip_tags($tp->toHTML($this->var['post_entry'], true, 'emotes_off, no_make_clickable', '', $pref['menu_wordwrap']));
        $post = $tp->text_truncate($post, varset($this->param['nfp_characters'], 120), varset($this->param['nfp_postfix'], '...'));

        return $post;
    }

$pref['menu_wordwrap'] is NULL

It is probably outdated code

tgtje commented 1 year ago

@Jimmi08 makes this sence ?

define("PRFLAN_110", "words longer than the length entered will be wrapped onto a new line"); define("PRFLAN_111", "Wrap long words in menu text");`

Jimmi08 commented 1 year ago

@tgtje yes, but is this still needed in responsive design? It should be wrapped to new line by css regardless this pref.

Or maybe I don't understand the purpose of this pref.

My worst problem was - if you have post with link inserted, html tags are stripped, then full url left there... and it was going out of place... Max number of characters will help if you split this link...

Will this pref help to this? If yes, wrapped is not good word, maybe split into a new line?

image

tgtje commented 1 year ago

Wording ?? overflow is general the same. Issue is : when url and design : nice if it follws the design, (breaks/overflows). But it would be different if long lines/url's are in a printable form...
Hard stuff : interesting link : this one