<div dir="ltr"><div class="gmail_extra">Thanks Petr. I thought I had grepped all that out. Guess I didn't do it from the top of the tree.<br><br></div><div class="gmail_extra">Updated patch attached.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Tue, May 5, 2015 at 5:15 AM, Petr Vobornik <span dir="ltr"><<a href="mailto:pvoborni@redhat.com" target="_blank">pvoborni@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 04/30/2015 07:43 PM, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks Kyle and Petr.<br>
<br>
Update patch attached.<br>
</blockquote>
<br></span>
Renaming the buttons also requires to update webui integration tests in ipatests/test_webui, quick search:<br>
  ipatests/test_webui/test_realmdomains.py:42,48<br>
  ipatests/test_webui/ui_driver.py:1221,1246,1464,1483<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Wed, Apr 29, 2015 at 7:59 AM, Kyle Baker <<a href="mailto:kybaker@redhat.com" target="_blank">kybaker@redhat.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
----- Original Message -----<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 04/27/2015 03:03 PM, Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
Fix for <a href="https://fedorahosted.org/freeipa/ticket/4926" target="_blank">https://fedorahosted.org/freeipa/ticket/4926</a><br>
<br>
Thanks,<br>
<br>
Gabe<br>
<br>
</blockquote>
<br>
PatternFly has new recommendations for terminology and wording [1]. I'm<br>
not entirely sure if the usage of 'save' here is good. PF defines 'edit'<br>
as the recommended term. The page doesn't say if 'save' is not<br>
recommended, though. Save seems to me as a confirmation of editing.<br>
</blockquote>
<br>
Yes I think save would be best here based on the message given.<br>
<br>
Thanks for checking out the Terminology screen!<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Kyle, could you advise what is the best term for reflecting user changes<br>
and for confirmation of this action?<br>
<br>
Technical notes:<br>
1. it would be better to add a new string and then use it in the button<br>
instead of having 'Save' text for '@i18n:buttons.update' definition.<br>
<br>
2. String changes in internal.py should be also reflected in<br>
install/ui/test/data/ipa_init.json (for static web ui demo).<br>
<br>
3. optional: in addition to text change, buttons and related actions<br>
could also be renamed (same reasons as in 1). It's more proper but much<br>
more complicated.<br>
<br>
<br>
[1]<br>
</blockquote>
<a href="https://www.patternfly.org/styles/terminology-and-wording/#action-labels" target="_blank">https://www.patternfly.org/styles/terminology-and-wording/#action-labels</a><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--<br>
Petr Vobornik<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Petr Vobornik<br>
</font></span></blockquote></div><br></div></div>