> On 2012-04-20 15:03:32, Anthony Carlucci wrote: > > /branches/bootstrap/rave-portal-resources/src/main/webapp/script/rave_person_profile.js, > > line 29 > > <https://reviews.apache.org/r/4813/diff/2/?file=103536#file103536line29> > > > > The function attached to the select event for the old jquery button is > > still needed. Please attach it to the tab click event for the bootstrap > > tabs.
That should have said "the old jquery tab" - Anthony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4813/#review7072 ----------------------------------------------------------- On 2012-04-20 13:15:13, Kristen Legacy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4813/ > ----------------------------------------------------------- > > (Updated 2012-04-20 13:15:13) > > > Review request for rave. > > > Summary > ------- > > RAVE-564: Update the Person Profile page > Added bootstrap tabs and form controls to profile. > > > Diffs > ----- > > > /branches/bootstrap/rave-portal-resources/src/main/webapp/WEB-INF/jsp/layouts/person_profile.jsp > 1328116 > > /branches/bootstrap/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/personProfile.jsp > 1328116 > > /branches/bootstrap/rave-portal-resources/src/main/webapp/script/rave_person_profile.js > 1328116 > > Diff: https://reviews.apache.org/r/4813/diff > > > Testing > ------- > > > Thanks, > > Kristen > >
