[
https://issues.apache.org/jira/browse/CONNECTORS-1196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14583307#comment-14583307
]
Karl Wright edited comment on CONNECTORS-1196 at 6/12/15 11:57 AM:
-------------------------------------------------------------------
[~kishorekumar]: Reformatting is fine but we cannot accept reformatting that
makes it impossible to read the embedded java code. It looks like your
reformat used Eclipse or something like it. Eclipse, though, is awful for
formatting JSP; it presumes that there is little logic in Java, because there
is no concept of bi-language indenting. In the MCF JSP's, we have both java
code and HTML, and each of these has an independent level of indent. Without
doing that, it is simply impossible to understand the java code
Can you describe why you need reformatting in order to proceed with this ticket?
I think we earlier agreed that it was a good idea to replace tabs with two
spaces, which would have basically preserved the independent indents. But the
formatting you checked in ties indenting levels from HTML to the Java, which I
don't think can work.
If I don't hear from you by tomorrow morning, I'll go ahead and revert the
trunk commit.
was (Author: [email protected]):
[~kishorekumar]: Reformatting is fine but we cannot accept reformatting that
makes it impossible to read the embedded java code. It looks like your
reformat used Eclipse or something like it. Eclipse, though, is awful for
formatting JSP; it presumes that there is little logic in Java, because there
is no concept of bi-language indenting. In the MCF JSP's, we have both java
code and HTML, and each of these has an independent level of indent. Without
doing that, it is simply impossible to understand the java code
Can you describe why you need reformatting in order to proceed with this ticket?
I think we earlier agreed that it was a good idea to replace tabs with two
spaces, which would have basically preserved the independent indents. But the
formatting you checked in tied indenting levels from HTML to the Java, which I
don't think can work.
If I don't hear from you by tomorrow morning, I'll go ahead and revert the
trunk commit.
> Proposal to update Manifold Crawler UI
> --------------------------------------
>
> Key: CONNECTORS-1196
> URL: https://issues.apache.org/jira/browse/CONNECTORS-1196
> Project: ManifoldCF
> Issue Type: Improvement
> Components: Framework core
> Affects Versions: ManifoldCF next
> Reporter: Kishore Kumar
> Assignee: Kishore Kumar
> Fix For: ManifoldCF next
>
> Attachments: Bandwidth Report.png, Index.jsp.png, List
> Repository.png, Report.png, View.png
>
>
> Hi,
> I have been working on a new User Interface for ManifoldCF.
> This UI is based on Bootstrap, jQuery and some other opensource scripts.
> Only the index.jsp will be loaded once and other page are rendered using AJAX
> call, making it more responsive and modern.
> Please find the attached screenshots.
> *Backward compatibility:*
> Since most of our tab comes from connectors, those tabs will be displayed as
> it is with no immediate attention required. (see sample screenshot attached)
> *What next?*
> In future we can add Charts to our reports where ever applicable, so that it
> will be more user friendly & informative for end users.
> Can make use of Web socket to notify exception or error to users as
> Notifications.
> *Browser compatibility*
> Has been tested with most latest browser and IE10, will try to add support
> for IE8 with acceptable degradation.
> Any suggestion, concern much appreciated.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)