Removing phpDoc comments

[Alexey Pyltsyn]

Before

After

Version 2.2.0

[Alexey Pyltsyn]

А на версии 2.1 комментарии не удаляются, но изначальная подсветка работает только для первой строки

[Yauheni Pakala]

Thanks for the feedback!

I will check this bug today.

P.S. Можете скинуть пример в текстовом виде, для ускорения воспроизведения?

[Alexey Pyltsyn]

Вот полный файл, кстати заметил сейчас ещё одну проблему - при копировании в начало каждой строки вставляется вертикальная строка (|), при отключенном номерах строк такого нет.

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;
use Doctrine\Common\Collections\ArrayCollection;

/**
 * @ORM\Entity
 */
class Post
{
    const NUM_ITEMS = 10;

    /**
     * @ORM\Id
     * @ORM\GeneratedValue
     * @ORM\Column(type="integer")
     */
    private $id;

    /**
     * @ORM\Column(type="string")
     */
    private $title;

    /**
     * @ORM\Column(type="string")
     */
    private $slug;

    /**
     * @ORM\Column(type="text")
     */
    private $content;

    /**
     * @ORM\Column(type="string")
     */
    private $authorEmail;

    /**
     * @ORM\Column(type="datetime")
     */
    private $publishedAt;

    /**
     * @ORM\OneToMany(
     *      targetEntity="Comment",
     *      mappedBy="post",
     *      orphanRemoval=true
     * )
     * @ORM\OrderBy({"publishedAt"="ASC"})
     */
    private $comments;

    public function __construct()
    {
        $this->publishedAt = new \DateTime();
        $this->comments = new ArrayCollection();
    }

    // getters and setters ...
}

[Alexey Pyltsyn]

@wcoder приветствую, как дела с этим багом? Куда копать для его исправления?

[Yauheni Pakala]

Здравствуйте, @lex111

Исправленный вариант уже сейчас можно попробовать здесь: https://raw.githubusercontent.com/wcoder/highlightjs-line-numbers.js/bugfixes/phpdoc/dist/highlightjs-line-numbers.min.js

В ближайшее время собираюсь выпустить обновление.

[Alexey Pyltsyn]

@wcoder проверил, работает отлично, спасибо 👍

[Alexey Pyltsyn]

Хотя есть небольшой баг, добавляется лишняя пустая строка, см. скриншоты ниже:

До включения плагина

После включения:

Может, из-за стилей подобное?

UPD: причем на всех языках, в HTML пустой пробел:

<div class="hljs-ln-line"> </div>```

[Yauheni Pakala]

@lex111

исправленный вариант:

https://raw.githubusercontent.com/wcoder/highlightjs-line-numbers.js/dev/dist/highlightjs-line-numbers.min.js

[Alexey Pyltsyn]

@wcoder круто, спасибо, скоро релиз?

[Alexey Pyltsyn]

@wcoder а можно еще одно улучшение, хотя оно не связанно с этим багом. В общем, можно проставлять класс для тега code, если нумерация строк кода включена, просто если блок кода состоит из одной строки, то нумерации строк нет, как раз чтобы отслеживать такую ситуацию нужен класс (в моем случае по умолчанию padding равен нулю, предполагая, что блок кода состоит из нескольких строк, соответственно, применяется плагин, и его стили, и если, блок кода состоит из одной строки, нужен небольшой padding), можно подобное сделать?

[Yauheni Pakala]

@lex111, вы можете включить нумерацию строк для одной строки, используя опцию singleLine:
https://github.com/wcoder/highlightjs-line-numbers.js#options

[Alexey Pyltsyn]

@wcoder да, но если этого не нужно, а отслеживать нумерацию как-то надо...

[Yauheni Pakala]

@lex111 используйте singleLine: true (нумерация будет отслеживаться) + стили (для скрытия нумерации)

[Yauheni Pakala]

@lex111 релиз 2.3 доступен в npm