[Freeipa-devel] [DOC] 0005 Updated chapter 4 - login into web UI

[Martin Basti]

On Fri, 2013-09-20 at 17:39 +0200, Petr Vobornik wrote:
> On 09/20/2013 04:06 PM, Martin Basti wrote:
> > Logging into web UI and configuring web browser sections were outdated
> >
> 
> Thanks for the path.
> 
> Here are my comments (some for existing issues):
> 
> 1. There are whitespace warnings when applying this patch.
Fixed
> 
> 2. PDF output is bad. Images are too big - right half is not displayed. 
> Several times there is image A with list item for image B below (image B 
> is on the next page with list item for image C).
> 
> I saw some 'missing image constraints warnings' during build - maybe 
> that's the cause.
More important is HTML, there is 440px width limit for images in PDF and
current images are not readable in PDF

> 3. Section 4.3.5: Don't know why it's called 'Simple authentication'. 
> Usually it's referred to as 'forms based authentication'.
Changed to form based auth

> 3a. Following sentence is misleading:
>    "the error first says to renew the Kerberos credentials or to
>     configure the browser to support Kerberos authentication."
> IIRC the previous dialog had different wording. The new one gives user 
> two options but it doesn't encourage user to 'renew Kerberos 
> credentials'. Also, you have deleted the first instruction but left a 
> second: "Then simply supply the UID and password for a configured 
> FreeIPA user." without any context. IMO it should be reworded.
Reworded

> 4. I think the entire section '4.3.6. Using the UI with Proxy Servers' 
> is incorrect. Using Web UI with proxy is not an easy thing to do. 
> <http://www.redhat.com/archives/freeipa-users/2013-August/msg00065.html>
Removed section

> 5. Old unused images should be deleted.
> 
Deleted + deleted unused xml which requires unused images
> 6. Section 4.4.1 (not sure if it's related to this patch) says: 
> "randomly selects up to 20 entries" that's not true. There is no 
> randomness. It selects: First record: ($PAGE_NUM * 20 +1), up to Last: 
> (($PAGE_NUM + 1) *20). When first index is 1.
Fixed

> 6a. LDAP search limit: the option name is --pkey-only not --pkey.
> 
Fixed

updated patch attached
-- 
Martin Basti
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0005-2-Updated-Chapter-4-login-into-Web-UI-section.patch
Type: text/x-patch
Size: 620820 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130930/91bc840c/attachment.bin>