<div dir="ltr">Ah - by mistake, I replied only to Bill. Cross-posting to the list as well. Sorry for the spam.<br><br>======<br><br>Hello there,<br><div class="gmail_quote"><div dir="ltr"><div><br>First - I found the current screencasts massively helpful in getting started, to be honest. They are something I really have not seen in most projects, and they contributed a lot to me making sense of the Keycloak codebase and usage scenarios. Kudos.<br></div><div><br></div><div>Sedond - I think it would be quite helpful to add additional modularization and also harmonize some configuration to make it more evident in how it is interpreted. Let me clarify the last bit here (I'll take themes as an example, but I believe that the same conclusion stands for other parts of the Keycloak codebase):<br><br><pre style="background-color:#ffffff;color:#000000;font-family:'Monaco';font-size:16,0pt"><span style="color:#808080;font-style:italic"># Import and extend definitions<br></span><span style="color:#000080;font-weight:bold">parent</span>=<span style="color:#008000;font-weight:bold">base<br></span><span style="color:#000080;font-weight:bold">import</span>=<span style="color:#008000;font-weight:bold">common/keycloak<br></span><span style="color:#000080;font-weight:bold">styles</span>=<span style="color:#008000;font-weight:bold">lib/patternfly/css/patternfly.css lib/zocial/zocial.css stylesheets/login.css stylesheets/yourOwn.css</span></pre><ol><li>The documentation and examples provides something roughly similar to the above.</li><li>Turning the attention to the "import" parameter, one could jump to the conclusion that there would be directory called "common/keycloak" and that this directory should contain a lib directory containing the styles css documents from the "styles" configuration.</li><li>Reading the codebase, it seems that the semantics of the "import" property is something completetly different. From the ExtendingThemeManager::loadTheme, I can see that the '/' is instead used as a list separator implying that we should attempt loading resources from several sources. (Snippet pasted below).<br></li></ol><pre style="background-color:#ffffff;color:#000000;font-family:'Monaco';font-size:16,0pt"><span style="color:#000080;font-weight:bold">if </span>(theme.getImportName() != <span style="color:#000080;font-weight:bold">null</span>) {<br> String[] s = theme.getImportName().split(<span style="color:#008000;font-weight:bold">"/"</span>);<br> themes.add(findTheme(s[<span style="color:#0000ff">1</span>], Theme.Type.<span style="font-style:italic">valueOf</span>(s[<span style="color:#0000ff">0</span>].toUpperCase())));<br>}</pre>So ... I would believe that the configuration in this case would be clearer on a Java Object, JSON or XML form, where one can provide somewhat better semantics than what is possible in a properties file (one could use a List of theme names instead of a single string value to be parsed and interpreted, for example).<br><br></div><div>Mind if I take a stab at implementing a suggestion here?<br></div><div><br></div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">2015-08-09 17:25 GMT+02:00 Bill Burke <span dir="ltr"><<a href="mailto:bburke@redhat.com" target="_blank">bburke@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Only plans right now are to separate our public SPIs and APIs from our<br>
private ones. This is a requirement by Red hat before we go into product.<br>
<br>
Also, a massive backlog of requirements and feature requests has made us<br>
rush documentation. The screencast videos haven't been updated since<br>
January. It is what it is. Over the next 3-6 months we will catch up<br>
on this stuff becuase we are required to before we go into Product.<br>
<br>
FYI, we already autogenerate REST docs.<br>
<span><br>
On 8/9/2015 7:38 AM, Lennart Jörelid wrote:<br>
> Hello all,<br>
><br>
> A month or so ago, I got curious about Keycloak. Downloaded, set up in a<br>
> dev environment, created some custom themes and took a look at the<br>
> codebase. I have a few questions, likely because I have missed some<br>
> developer documentation:<br>
><br>
</span>> * *Codebase concepts*: I frequently try to structure codebases to<br>
<span>> highlight its big concepts. For example, if we consider 'themes' to<br>
> be such a concept in KeyCloak we might create a folder called<br>
> 'themes", with some project wihtin it: (themes-model, themes-spi,<br>
> themes-impl-jpa, themes-impl-freemarker, ....). Is there a<br>
> description of the codebase structure or concepts currently?<br>
> ("mini-SAD")<br>
</span>> * *Codebase javadoc:* Do we have a policy for JavaDoc'ing the<br>
<span>> Model/API/SPI but perhaps not the implementation classes, other than<br>
> with implementation details?<br>
</span>> * *Configuration:* Some of the descriptions in the docbook are really<br>
<span>> good, and some are more shallow. If we create a standard way of<br>
> configuring the parts of keycloak, we could likely generate standard<br>
> setup/configuration documentation (somewhat similar to maven plugins<br>
> where certain parts of a site documentation is generated from<br>
> annotations or JavaDocs). Are there such plans?<br>
><br>
><br>
> --<br>
><br>
> --<br>
> +==============================+<br>
> | Bästa hälsningar,<br>
> | [sw. "Best regards"]<br>
> |<br>
> | Lennart Jörelid<br>
> | EAI Architect & Integrator<br>
> |<br>
> | jGuru Europe AB<br>
> | Mölnlycke - Kista<br>
> |<br>
</span>> | <a href="mailto:Email%3Alj@jguru.se" target="_blank">Email:lj@jguru.se</a> <mailto:<a href="mailto:lj@jguru.se" target="_blank">lj@jguru.se</a>><br>
> | URL:<a href="http://www.jguru.se" rel="noreferrer" target="_blank">www.jguru.se</a> <<a href="http://www.jguru.se" rel="noreferrer" target="_blank">http://www.jguru.se</a>><br>
<span>> | Phone<br>
> | (skype): jgurueurope<br>
> | (intl): <a href="tel:%2B46%20708%20507%20603" value="+46708507603" target="_blank">+46 708 507 603</a><br>
> | (domestic): 0708 - 507 603<br>
> +==============================+<br>
><br>
><br>
><br>
</span>> _______________________________________________<br>
> keycloak-dev mailing list<br>
> <a href="mailto:keycloak-dev@lists.jboss.org" target="_blank">keycloak-dev@lists.jboss.org</a><br>
> <a href="https://lists.jboss.org/mailman/listinfo/keycloak-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/keycloak-dev</a><br>
><br>
<span><font color="#888888"><br>
--<br>
Bill Burke<br>
JBoss, a division of Red Hat<br>
<a href="http://bill.burkecentral.com" rel="noreferrer" target="_blank">http://bill.burkecentral.com</a><br>
_______________________________________________<br>
keycloak-dev mailing list<br>
<a href="mailto:keycloak-dev@lists.jboss.org" target="_blank">keycloak-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/keycloak-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/keycloak-dev</a><br>
</font></span></blockquote></div></div></div></div></div><br>-- <br><div class="gmail_signature"><span style="font-family:monospace;font-size:medium"><pre>--
+==============================+
| Bästa hälsningar,
| [sw. "Best regards"]
|
| Lennart Jörelid
| EAI Architect & Integrator
|
| jGuru Europe AB
| Mölnlycke - Kista
|
| Email: <a href="mailto:lj@jguru.se" target="_blank">lj@jguru.se</a>
| URL: <a href="http://www.jguru.se" target="_blank">www.jguru.se</a>
| Phone
| (skype): jgurueurope
| (intl): +46 708 507 603
| (domestic): 0708 - 507 603
+==============================+</pre></span></div>
</div>