Commit 4c561994 authored by Glen Ditchfield's avatar Glen Ditchfield 🐛 Committed by Glen Ditchfield
Browse files

Fix keyboard focus-change order

With version 5.19.3, using the keyboard to move between the Calendar and
Summary fields is a bit off.

* Generally the user must press `Tab` or `Shift-Tab` twice.  Removing
  some properties from the Category label fixes that, and also removes
  an underline from the 'd' in the 'Calendar' label.

* If the Attachments tab is visible, the user must press the key three
  times;  focus moves to the attachment list before reaching the final
  destination.  Adding a `setTabOrder()` fixes that.

BUG: 331543
parent 5a02a1c1
Pipeline #159288 failed with stage
in 1 minute and 31 seconds
......@@ -524,6 +524,7 @@ void IncidenceAttachment::setupAttachmentIconView()
auto layout = new QGridLayout(mUi->mAttachmentViewPlaceHolder);
layout->setContentsMargins(0, 0, 0, 0);
layout->addWidget(mAttachmentView);
QWidget::setTabOrder(mUi->mAddButton, mAttachmentView);
}
// void IncidenceAttachmentEditor::addAttachment( KCalendarCore::Attachment *attachment )
......
......@@ -52,15 +52,9 @@
<property name="text">
<string comment="@label:listbox The calendar to which the todo or event must be added">Calen&amp;dar:</string>
</property>
<property name="textFormat">
<enum>Qt::PlainText</enum>
</property>
<property name="alignment">
<set>Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter</set>
</property>
<property name="textInteractionFlags">
<set>Qt::LinksAccessibleByKeyboard|Qt::LinksAccessibleByMouse</set>
</property>
</widget>
</item>
<item row="0" column="1">
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment