logiclord / epubReader

epubReader is a standalone fluid infusion component. This project is about implementing a web based ePub reader component based on open web technologies for infusion framework

Home Page: http://webbasedepubreader.blogspot.in

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

More file structure improvements

yzen opened this issue · comments

Hi Gaurav,

  • I was wondering if it is necessary to duplicate some of the templates specific to UIOptions. I noticed you have non-epub templates in html/uiOptions folder that are also present in the "proper" location infusion/components/uiOptions/html.
  • Also, you still have JSZip and Infusion at the top level, rather than in lib folder, I think it makes sense to group those in there.

Hi Yura,

I will be cleaning of directory structure in my next commit. I kept most of
those templates while experimenting with UI Options. I really don't need
layout and link template while I am using same Textual template as in
infusion, rest are of my own.

I will ping you when I am done so that you can review most updated code.

On Thu, Jul 5, 2012 at 7:26 PM, Yura Zenevich <
reply@reply.github.com

wrote:

Hi Gaurav,

  • I was wondering if it is necessary to duplicate some of the templates
    specific to UIOptions. I noticed you have non-epub templates in
    html/uiOptions folder that are also present in the "proper" location
    infusion/components/uiOptions/html.
  • Also, you still have JSZip and Infusion at the top level, rather than in
    lib folder, I think it makes sense to group those in there.

Reply to this email directly or view it on GitHub:
https://github.com/logiclord/web-based-epub-reader/issues/11

Regards,

Gaurav Aggarwal
Final year Student,
Delhi College of Engineering

Great, I just had some time to check it out and give some suggestions. I like what you did with the component structure in the epub reader code, by the way!

I have changed the file structure. Please confirm about new file structure.