Bug 80387 - [EasyHack] extend lint-ui.py to check for UI title labels
Summary: [EasyHack] extend lint-ui.py to check for UI title labels
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Jorendc
QA Contact:
URL:
Whiteboard: EasyHack
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-23 10:48 UTC by Thomas Arnhold
Modified: 2014-06-23 12:55 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Python script to check title cases (3.88 KB, text/x-python-script)
2014-06-23 11:07 UTC, Jorendc
Details

Description Thomas Arnhold 2014-06-23 10:48:47 UTC
We have lint-ui.py, which checks for several settings of ui files, so that they have consistent borders and so on. See:

http://cgit.freedesktop.org/libreoffice/core/tree/bin/lint-ui.py

The idea is to extend the script to ensure that all title words are uppercase. There was a commit, which corrected those: 171efcb7acf367e8db53694489815452d2a9894a

But for the future it would be nice if we could warn about those cases.

The titles are inside node: <child type="label">
Comment 1 Jorendc 2014-06-23 11:04:25 UTC
I did write a python script to check for titles. I'm currently working to check normal labels too (not only title labels).
I'll attach the script used to this bug report.

Afterwards I'll have a look at this lint-ui.py to implement it over there.

Kind regards,
Joren
Comment 2 Jorendc 2014-06-23 11:07:17 UTC
Created attachment 101573 [details]
Python script to check title cases

Still some manual work/review to do. It only splits words by space. Words surrounded by brackets or with other punctuation marks will need different approach.

Kind regards,
Joren
Comment 3 Thomas Arnhold 2014-06-23 12:55:01 UTC
Ah, great! So I'll assign you to this bug.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.