D14435: Fix KTimeComboBox input mask for AM/PM times
    Laurent Montel 
    noreply at phabricator.kde.org
       
    Mon Jul 30 07:06:53 BST 2018
    
    
  
mlaurent requested changes to this revision.
mlaurent added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> ktimecomboboxtest.cpp:198
> +    QString mask = m_combo->lineEdit()->inputMask();
> +    QCOMPARE(mask.contains(QLatin1String("aa")), true);
> +    delete m_combo;
Could you provide another case where mask will not contains "aa" so we can be sure that code works
Thanks.
REPOSITORY
  R236 KWidgetsAddons
REVISION DETAIL
  https://phabricator.kde.org/D14435
To: glennw, cfeck, mlaurent
Cc: ngraham, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180730/a430a21d/attachment-0001.html>
    
    
More information about the Kde-frameworks-devel
mailing list