lhousehold/oncourse

Textbook-centred Teaching and Learning Interface

oncourse's Issues

This should be an id not a class.

Me and howard were confused by this.

    <div class="pos_all cppage_top" data-page-number="#{pagenumber}">

we removed it because we couldn't find a style for it.
It seems cppage_top is used only in your nav button code but it makes more sence to be an ID.
We reverted put it back like so but I still think you should change it to an ID.

database interface

I want to make a change to the database interface so if you haven't used it yet, then don't. Or if you do then expect a small change to your code while looping through the results.

Removing hard coded CPID when bring up a course package

I wanted to hard code some different course package id's to the side bar items so we could host
multiple course packages (right now it's hard coded to 1 every time it tries to draw a cp).

I'm getting some issues concerning types and passing values. This is what's happening right now:

Here's the error:

javax.el.ELException: /cppage.xhtml @12,96 value="#{coursePackageDisplay.pageFetch(1,pagenumber)}": Method not found: class com.oncourse.CoursePackageDisplay.pageFetch(java.lang.Long, java.lang.String)

It's basically saying that it can't find a method called pageFetch that takes a Long and a String. The only one that exists takes two integers so it says the one we're looking for doesn't exist.

Similarly:

javax.el.ELException: /cppage.xhtml @12,103 value="#{coursePackageDisplay.pageFetch(cpnumber,pagenumber)}": Method not found: class com.oncourse.CoursePackageDisplay.pageFetch(java.lang.String, java.lang.String)

I don't know if Java lets you pass a Long for an Int but I somehow doubt it. I don't know why it worked before or what I changed to make it think that the pagenumber variable is now a string. Very strange.

The safest thing to do here might be to always expect Strings if you are a Java method invoked by an xhtml page. What do you guys think?

text highlighting when clicking right and left arrow buttons

Luke I know you already know this issue is a thing, here is one way that fixes it. Feel free to use whatever. But this is not standard yet so all flags are needed for it work on all browsers.
.noselect {
-webkit-touch-callout: none;
-webkit-user-select: none;
-khtml-user-select: none;
-moz-user-select: none;
-ms-user-select: none;
user-select: none;
}