<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>309960</bug_id>
          
          <creation_ts>2012-11-12 13:17:16 +0000</creation_ts>
          <short_desc>HTML in window titles not escaped in tabbox</short_desc>
          <delta_ts>2012-11-27 19:57:34 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>kwin</product>
          <component>tabbox</component>
          <version>4.9.2</version>
          <rep_platform>Gentoo Packages</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>data:text/html,&lt;i&gt;Hi&lt;/i&gt; &lt;u&gt;there&lt;/u&gt;</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>4.9.4</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>wrtlprnft</reporter>
          <assigned_to name="KWin default assignee">kwin-bugs-null</assigned_to>
          
          
          <cf_commitlink>http://commits.kde.org/kde-workspace/73fc361dfb05e6e76b29c43406a29d32b4991a8f</cf_commitlink>
          <cf_versionfixedin>4.9.4</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      <flag name="ReviewRequest"
          id="436"
          type_id="14"
          status="+"
          setter="mgraesslin"
    />

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1314795</commentid>
    <comment_count>0</comment_count>
    <who name="">wrtlprnft</who>
    <bug_when>2012-11-12 13:17:16 +0000</bug_when>
    <thetext>When using alt-tab with windows whose title contains HTML tags, those tags are interpreted when displaying the window title in the tabbox.

See http://wrtlprnft.ath.cx/kwin_html.png for an example (I deleted ~/.kde* before taking this snapshot).

Reproducible: Always

Steps to Reproduce:
1. Point konqueror to the address data:text/html,&lt;i&gt;Hi&lt;/i&gt; &lt;u&gt;there&lt;/u&gt;
2. Press alt-tab, cycle through windows until the konqueror window is active
Actual Results:  
konqueror&apos;s title is displayed with formatting

Expected Results:  
the title should be displayed in the same way as it&apos;s displayed in the window&apos;s title bar.

This can obviously reproduced by other means than konqueror (konsole/xterm title, :set titletext in gvim, opening a file with angle brackets in its name, …).

Probably related to 293657</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1314867</commentid>
    <comment_count>1</comment_count>
    <who name="Martin Flöser">mgraesslin</who>
    <bug_when>2012-11-12 17:26:14 +0000</bug_when>
    <thetext>quite related to the pointed out bug (that prevented me from googling for the solution - thanks) and easy to fix. Only problem is that I would prefer a general solution over fixing each of the QML files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1317686</commentid>
    <comment_count>2</comment_count>
    <who name="Martin Flöser">mgraesslin</who>
    <bug_when>2012-11-23 07:38:50 +0000</bug_when>
    <thetext>https://git.reviewboard.kde.org/r/107431/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1318927</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Flöser">mgraesslin</who>
    <bug_when>2012-11-27 19:56:53 +0000</bug_when>
    <thetext>Git commit d88a57be3885402e857140305ca0edbdbe4f7342 by Martin Gräßlin.
Committed on 23/11/2012 at 08:23.
Pushed by graesslin into branch &apos;KDE/4.9&apos;.

Escape HTML command in window caption before passing to QML

Escaping is performed in the model instead of the UI as each of the QML
files needed to be fixed and it is likeley that the issue would come up
again. In the model it&apos;s hopefully fixed for good.
FIXED-IN: 4.9.4
REVIEW: 107431

M  +9    -2    kwin/tabbox/clientmodel.cpp

http://commits.kde.org/kde-workspace/d88a57be3885402e857140305ca0edbdbe4f7342</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1318928</commentid>
    <comment_count>4</comment_count>
    <who name="Martin Flöser">mgraesslin</who>
    <bug_when>2012-11-27 19:57:34 +0000</bug_when>
    <thetext>Git commit 73fc361dfb05e6e76b29c43406a29d32b4991a8f by Martin Gräßlin.
Committed on 23/11/2012 at 08:23.
Pushed by graesslin into branch &apos;master&apos;.

Escape HTML command in window caption before passing to QML

Escaping is performed in the model instead of the UI as each of the QML
files needed to be fixed and it is likeley that the issue would come up
again. In the model it&apos;s hopefully fixed for good.
FIXED-IN: 4.9.4
REVIEW: 107431

M  +9    -2    kwin/tabbox/clientmodel.cpp

http://commits.kde.org/kde-workspace/73fc361dfb05e6e76b29c43406a29d32b4991a8f</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>