[Libreoffice-ux-advise] [Bug 146468] Use same border preset tooltips between toolbar and sidebar

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=146468

Heiko Tietze <heiko.tietze at documentfoundation.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
                 CC|libreoffice-ux-advise at lists |heiko.tietze at documentfounda
                   |.freedesktop.org            |tion.org,
                   |                            |mentoring at documentfoundatio
                   |                            |n.org
             Status|UNCONFIRMED                 |NEW
           Keywords|needsUXEval                 |difficultyBeginner,
                   |                            |easyHack, skillCpp, topicUI

--- Comment #3 from Heiko Tietze <heiko.tietze at documentfoundation.org> ---
The toolbar button uses include/svx/strings.hrc starting with #define
RID_SVXSTR_TABLE_PRESET_NONE while the sidebar is defined in a ui file with
tooltips for each item. I suggest to assign the RID_SVXSTR* tooltips per code
like 

sc/source/ui/sidebar/CellBorderStyleControl.cxx

#include <svx/strings.hrc>
#include <svx/dialmgr.hxx>

void CellBorderStylePopup::Initialize()
{
    mxTBBorder1->connect_clicked ( LINK(this, CellBorderStylePopup,
TB1SelectHdl) );
    mxTBBorder1->set_item_tooltip_text(0,
SvxResId(RID_SVXSTR_TABLE_PRESET_NONE));
    mxTBBorder1->set_item_tooltip_text(1, SvxResId(
    ...

Perhaps it makes sense to clean up tooltip from the ui file to remove
confusion.

-- 
You are receiving this mail because:
You are on the CC list for the bug.