diff options
author | Andrew Dyukov <adyukov@productengine.com> | 2010-08-09 15:14:45 +0300 |
---|---|---|
committer | Andrew Dyukov <adyukov@productengine.com> | 2010-08-09 15:14:45 +0300 |
commit | b93474301489d2192cc9b7c8a675b55f49c9577b (patch) | |
tree | f23c6a2e3e5044cd6ef1a7b326d003dba2cf3dc0 /indra/newview/skins | |
parent | ff6eccd32db0cf57e16db8941b25a12dbb64b9fa (diff) |
EXT-8010 ADDITIONAL FIX Fixed problems with avatar links underlining.
There were two problems:
1. Underlining broke when avatar's first and second name were on different lines.
2. There was no underline on hover for avatar miniinspector links in plaintext IM.
- First problem was caused by calling LLOnHoverChangeableTextSegment::draw() for the same segment twice- for first and second name that were
on different lines, while handleHover() was called only once. So handleHover() was called -> text was underlined -> first part of segment was
drawn underlined -> its draw set style back to normal -> second part of segment was drawn without underlining.
Fixed this by setting style back to normal only when drawing the last part of the segment.
- Second problem was caused by unusual way of appending link to text in chat history.
Changed it so that LLTextBase::appendText() now receives link not inside style params, but directly.
Also added "/inspect" ending to check in LLUrlEntryAgent::underlineOnHoverOnly().
Reviewed by Richard Nelson at https://codereview.productengine.com/secondlife/r/833/
--HG--
branch : product-engine
Diffstat (limited to 'indra/newview/skins')
0 files changed, 0 insertions, 0 deletions