>-----Original Message-----
>From: Jasha Joachimsthal [mailto:[email protected]]
>Sent: Tuesday, November 08, 2011 4:42 AM
>To: [email protected]
>Subject: Re: svn commit: r1199062 - in /incubator/rave/trunk/rave-portal-
>resources/src/main/webapp/WEB-INF: dispatcher-servlet.xml jsp/ jsp/views/
>views/ web.xml
>
>Good thing to make clear it's the folder for jsp as presentation layer, but
>why jsp/views and not just jsp?

I'm planning to add a bunch of other folders under WEB-INF/jsp as well -- 
although we still may want to change "views" to something else at some point.  
Some of the other folders I'll be adding will be:

WEB-INF/jsp/common -- place for common stuff like header and footer code
WEB-INF/jsp/includes -- place for includes like JSP taglib definitions
WEB-INF/jsp/templates -- place for tiles base layout template files

What will end up left in the current "views" folder will be basically just the 
content that makes up the body of each page (the content between the header and 
footer).  

Anyone have and good suggestions for what to change the folder name to if we 
don't stick with "views"?  I've used "tiles" in the past since they end up 
being little "tiles" of content but I'm not sure I really like that either 
since it might be confused with tiles proper.

>On 8 November 2011 03:11, <[email protected]> wrote:
>
>> Author: jcian
>> Date: Tue Nov  8 02:11:19 2011
>> New Revision: 1199062
>>
>> URL: http://svn.apache.org/viewvc?rev=1199062&view=rev
>> Log:
>> Shuffling around where the views are located in preparation for adding
>> tiles - supports RAVE-343.
>>
>> Added:
>>    incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/jsp/
>>
>>  incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/jsp/views/
>>      - copied from r1199057,
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/views/
>> Removed:
>>
>>  incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/views/
>> Modified:
>>
>>  incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/dispatcher-servlet.xml
>>
>>  incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/web.xml
>>
>> Modified:
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/dispatcher-servlet.xml
>> URL:
>> http://svn.apache.org/viewvc/incubator/rave/trunk/rave-portal-
>resources/src/main/webapp/WEB-INF/dispatcher-
>servlet.xml?rev=1199062&r1=1199061&r2=1199062&view=diff
>>
>>
>===========================================================
>===================
>> ---
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/dispatcher-servlet.xml
>> (original)
>> +++
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/dispatcher-servlet.xml
>> Tue Nov  8 02:11:19 2011
>> @@ -39,7 +39,7 @@
>>
>>     <!-- Resolves view names to protected .jsp resources within the
>> /WEB-INF/views directory -->
>>     <bean
>>
>class="org.springframework.web.servlet.view.InternalResourceViewResolver
>">
>> -        <property name="prefix" value="/WEB-INF/views/"/>
>> +        <property name="prefix" value="/WEB-INF/jsp/views/"/>
>>         <property name="suffix" value=".jsp"/>
>>     </bean>
>>
>>
>> Modified:
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/web.xml
>> URL:
>> http://svn.apache.org/viewvc/incubator/rave/trunk/rave-portal-
>resources/src/main/webapp/WEB-
>INF/web.xml?rev=1199062&r1=1199061&r2=1199062&view=diff
>>
>>
>===========================================================
>===================
>> ---
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/web.xml
>> (original)
>> +++
>> incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-
>INF/web.xml
>> Tue Nov  8 02:11:19 2011
>> @@ -74,6 +74,6 @@
>>
>>     <error-page>
>>         <exception-type>java.lang.Throwable</exception-type>
>> -        <location>/WEB-INF/views/error.jsp</location>
>> +        <location>/WEB-INF/jsp/views/error.jsp</location>
>>     </error-page>
>>  </web-app>
>>
>>
>>

Reply via email to