Foreground color is set to background color if sequence lists background first #17

Activity

1. ellisgl commented on Aug 1, 2018

   

   ellisgl

   on Aug 1, 2018

   More actions

   ^[[0m is reset, so the colors should back to white-on-black, then convert at the next sequence...
2. fctr commented on Feb 20, 2020

   

   fctr

   on Feb 20, 2020

   Last edited by fctr

   More actions

       public function convert($text)
       {
           // remove cursor movement sequences
           $text = preg_replace('#\e\[(K|s|u|2J|2K|\d+(A|B|C|D|E|F|G|J|K|S|T)|\d+;\d+(H|f))#', '', $text);
           // remove character set sequences
           $text = preg_replace('#\e(\(|\))(A|B|[0-2])#', '', $text);
   
           $text = htmlspecialchars($text, PHP_VERSION_ID >= 50400 ? ENT_QUOTES | ENT_SUBSTITUTE : ENT_QUOTES, $this->charset);
   
           // carriage return
           $text = preg_replace('#^.*\r(?!\n)#m', '', $text);
   
           // Add the follow line
           $text = preg_replace('/m\e\[/', ';', $text);
   
   
   
3. fabpot

   added a commit that references this issue on Mar 15, 2025

   bug #35 Handle multiple codes in sequences (jacklul)

   ...

   f9a7bea
4. fabpot

   closed this as completedon Mar 15, 2025