Skip to content
This repository has been archived by the owner on Dec 9, 2022. It is now read-only.

alerts: inconsistent capitalization #14

Open
dperpeet opened this issue Mar 22, 2016 · 1 comment
Open

alerts: inconsistent capitalization #14

dperpeet opened this issue Mar 22, 2016 · 1 comment

Comments

@dperpeet
Copy link
Contributor

Most conditions and phrases for setroubleshoot fix solutions start with a capital letter, but some don't.
Example:

If you believe that SetroubleshootF should be allowed write access on the selinux directory by default.
You should report this as a bug. You can generate a local policy module to allow this access.

In the Cockpit UI it looks strange to have the text visible if some lines start with a lower case letter:

If you believe SetroubleshootF tried to disable SELinux.
you may be under attack by a hacker, since confined applications should never need this access.

I believe this should be consistent and proposed a fix in #11 where I hopefully got all plugin cases.

@dperpeet
Copy link
Contributor Author

screenshot of the work in progress branch where you can see the inconsistency:
screenshot from 2016-03-22 11-43-19

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant