On 05/05/2015 02:36 PM, Gabe Alford wrote:
Thanks Petr. I thought I had grepped all that out. Guess I didn't do it
from the top of the tree.

Updated patch attached.


Pushed to:
master: d1a0474d1851cd54a1fa7bdf4a0dc0ed452a0090
ipa-4-1: 5ac5564227757dfc367ca74fe86e99114d5b582b

On Tue, May 5, 2015 at 5:15 AM, Petr Vobornik <pvobo...@redhat.com> wrote:

On 04/30/2015 07:43 PM, Gabe Alford wrote:

Thanks Kyle and Petr.

Update patch attached.

Renaming the buttons also requires to update webui integration tests in
ipatests/test_webui, quick search:

On Wed, Apr 29, 2015 at 7:59 AM, Kyle Baker <kyba...@redhat.com> wrote:

----- Original Message -----

On 04/27/2015 03:03 PM, Gabe Alford wrote:


Fix for https://fedorahosted.org/freeipa/ticket/4926



PatternFly has new recommendations for terminology and wording [1]. I'm
not entirely sure if the usage of 'save' here is good. PF defines 'edit'
as the recommended term. The page doesn't say if 'save' is not
recommended, though. Save seems to me as a confirmation of editing.

Yes I think save would be best here based on the message given.

Thanks for checking out the Terminology screen!

Kyle, could you advise what is the best term for reflecting user changes
and for confirmation of this action?

Technical notes:
1. it would be better to add a new string and then use it in the button
instead of having 'Save' text for '@i18n:buttons.update' definition.

2. String changes in internal.py should be also reflected in
install/ui/test/data/ipa_init.json (for static web ui demo).

3. optional: in addition to text change, buttons and related actions
could also be renamed (same reasons as in 1). It's more proper but much
more complicated.



Petr Vobornik

Petr Vobornik

Petr Vobornik

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to