iRex Forum Forum Index iRex Forum

 
 FAQFAQ   SearchSearch   UsergroupsUsergroups   RegisterRegister 
 ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

Fullscreen mode ipdf
Goto page 1, 2  Next
 
Post new topic   Reply to topic    iRex Forum Forum Index -> iPDF Viewer
View previous topic :: View next topic  
Author Message
Matthijs
iRex Employee
iRex Employee


Joined: 18 Jul 2006
Posts: 948
Location: Eindhoven

PostPosted: Mon Jul 09, 2007 2:06 pm    Post subject: Fullscreen mode ipdf Reply with quote

We're contemplating asking the community to develop a fullscreen patch on top of iPDF that can be integrated in the mainstream. Reason is that something like that is already mostly available, but integrating it in its current form is not possible (see spec below).

We'd like to get more interaction with the community and actually get stuff merged back into the mainstream. Problem is: we don't have the time to adapt the currently available stuff to suit our needs and this is why we're asking you guys. I hope you won't take this the wrong way.

Ok, here we go:
Specification would be (different from what is now available from enhanced-ipdf):

1. Toolbar icon to switch to fullscreen
2. Short press of the UP key to exit fullscreen mode

Fullscreen defined as overlapping toolbar and pageBar by setting gtk_window_fullscreen()

We would be providing a patched IPC lib and an icon.

Would this be something you are interested in? Maybe Rincewind would like to create this from his existing patch?
Back to top
View user's profile Send private message
jharker



Joined: 25 Apr 2007
Posts: 281
Location: Rochester, NY, USA

PostPosted: Mon Jul 09, 2007 11:54 pm    Post subject: Reply with quote

I'd be happy to work on this a little bit. Regarding #2, that change would be trivial, just redefining what button does what. Regarding #1, I don't really know anything about how the toolbar icons are implemented... so it would be great if someone who knows more than me (rincewind? jayk?) could take the lead on that.

Aside from this, what other ipdf improvements are you interested in? The power-saving mods? Some of rincewind's shortcuts? Personally, I find rincewind's quick-erase function absolutely fabulous, and even in the absence of all else, I would advocate including it.

In addition, there are several "bugs" with the hide-the-toolbar patch that need to be addressed. These include:
  • In fullscreen mode, for some reason the toolbar auto-hides when you toggle continuous mode, even though it is not supposed to (in the code). I don't know why this happens.
  • In fullscreen mode, the toolbar auto-hides after some other button presses, but I don't recall exactly what. None of these are by design, and we should figure out why that happens.
  • We need to decide what is proper behavior for the toolbar. For example, should the toolbar auto-hide after "zoom" is selected? After "erase" or "scribble" is selected?
  • The screen refresh could be slimmed down; in some situations two refreshes are performed in a row; that should be reduced to one.

Also, Matthijs, you said:
Quote:
Fullscreen defined as overlapping toolbar and pageBar by setting gtk_window_fullscreen()

...however, that's not quite how fullscreen works, nor is it possible to do it that way. The current ipdf writes the page image directly to the screen buffer, OVER whatever is already there. The screen area written to is hard-coded to exclude the toolbar area. My fullscreen patch actually changes the writable screen area limits defined in the code.

What I'm trying to say is, just setting gtk_window_fullscreen will hide the toolbar and PageBar (and the code does that), but additional changes are necessary to make ipdf paint the whole screen with a page image. Is that okay?

All in all this sounds like a cool project. I'm on vacation for the next week, and then moving my apartment after that, but I'll contribute all I can!
Back to top
View user's profile Send private message
realityloop



Joined: 20 Apr 2007
Posts: 54

PostPosted: Tue Jul 10, 2007 12:43 am    Post subject: Reply with quote

I'd like to suggest that when in fullscreen mode with the toolbar displayed that any toolbar icon that is selected then auto hides the toolbar, I'd see this as especially relevant behaviour if you have already zoomed your document for best viewing at full screen.

I also agree that quick erase should be part of it, and the powersaving features, though the powersaving features should possibly be an optional setting.
Back to top
View user's profile Send private message
jharker



Joined: 25 Apr 2007
Posts: 281
Location: Rochester, NY, USA

PostPosted: Tue Jul 10, 2007 5:05 am    Post subject: Reply with quote

Okay, I realized I had a couple of other questions...

Matthijs wrote:
We would be providing a patched IPC lib and an icon.

So, I realized I may not have understood this... do you mean that you'd provide a modified toolbar with a 'fullscreen' icon, and we'd simply have to modify ipdf to read the 'fullscreen' icon's input and react accordingly? Or would we need to actually modify the toolbar ourselves?

I don't know how to modify the toolbar (that's what I was asking for help with in my previous post), but if I'm given a toolbar with the fullscreen icon already added, I can easily change ipdf to react appropriately.

Second question: what about zoom? The way ipdf is implemented right now, the zoom level does NOT change when fullscreen mode is turned on. Instead, more of the page is made visible. This minimizes processing time, making it very quick to toggle fullscreen mode on or off.

There have been some suggestions about how to modify fullscreen to deal more intuitively with zoom, but they would require some work, and turning fullscreen on/off would be slower.

Another question... should ipdf default to fullscreen mode, or normal mode?

@realityloop - I would be in favor of including powersaving, but I seem to remember Matthijs suggesting that slowing down the CPU can make some kernel drivers unstable. Unless those problems have been fixed, I doubt we can include it... iRex can't include instabilities in an official release. Matthijs, am I right about this, or am I misremembering?

Matthijs, perhaps it would be helpful for us to work on creating a complete list of exactly what features you want added?
Back to top
View user's profile Send private message
Matthijs
iRex Employee
iRex Employee


Joined: 18 Jul 2006
Posts: 948
Location: Eindhoven

PostPosted: Tue Jul 10, 2007 7:20 am    Post subject: Reply with quote

Powersaving will be done separately in the powerMgr, on a system level. We're still looking at including this in v2.11 (it's lying on the shelf, just have to assess the stability issues if any).

You don't need to modify the toolbar to add an icon, only the IPC lib needs a small change (and the registry). We'll provide the changes for that, so you only need to call the general toolbar icon functions with a new IconID.

The other functions of the enhanced ipdf are just not compatible with our UI and will never be approved by the concept developers for inclusion in a release version.
Shortcut buttons are a big no-no, especially if they are supposed to be assigned to another function (like the up/down arrow and confirm keys).

As for zoom and fullscreen: that is indeed an interesting one... I'm inclined to say that I'd like to at least zoom-to-fit fullscreen when the original one was zoom-to-fit.

So for now I'd like to add fullscreen only.

If deleting all scribbles on a page is to be added, it would need to use a toolbar icon. Maybe we can use the trashcan for that, as the Notes app does...
Back to top
View user's profile Send private message
Zeppy



Joined: 07 Jun 2007
Posts: 37

PostPosted: Tue Jul 10, 2007 12:08 pm    Post subject: Reply with quote

jharker wrote:

Another question... should ipdf default to fullscreen mode, or normal mode?


Don't know if possible, but if it is, we could include an option to choose, or even maybe decide that if at the application startup the pen is in its folder then the default is fullscreen, normal otherwise...
Back to top
View user's profile Send private message
Matthijs
iRex Employee
iRex Employee


Joined: 18 Jul 2006
Posts: 948
Location: Eindhoven

PostPosted: Tue Jul 10, 2007 12:24 pm    Post subject: Reply with quote

The ideal situation would be that iPDF stores this state inside the manifest.xml. But for now the startup behavior would be to include the toolbar and pageBar (e.g. don't remember the fullscreen state).
Back to top
View user's profile Send private message
realityloop



Joined: 20 Apr 2007
Posts: 54

PostPosted: Tue Jul 10, 2007 1:40 pm    Post subject: Reply with quote

It would be possible to have the the toolbar collapse sideways to a single icon (or half icon) as a solution to not using shortcut buttons while still gaining maximum screen real estate for reading.

I'd mock up what I'm thinking but I'm unable to find a clean screen grab of iPDF.

If you can link me a clean screenshot I'll happily mock up to show how I'd see the different states looking.
Back to top
View user's profile Send private message
jharker



Joined: 25 Apr 2007
Posts: 281
Location: Rochester, NY, USA

PostPosted: Tue Jul 10, 2007 3:27 pm    Post subject: Reply with quote

Matthijs,

Ah, that sounds good! Let's see:

Matthijs wrote:
You don't need to modify the toolbar to add an icon, only the IPC lib needs a small change (and the registry). We'll provide the changes for that, so you only need to call the general toolbar icon functions with a new IconID.

Yes, just modifying ipdf to use a new button will be easy. That's great.

Matthijs wrote:
The ideal situation would be that iPDF stores this state inside the manifest.xml. But for now the startup behavior would be to include the toolbar and pageBar (e.g. don't remember the fullscreen state).

Actually, the current implementation of fullscreen stores the fullscreen state of the document when it closes. My question was, how should the viewer handle opening a document that has no history. It sounds like you want it to open in non-fullscreen by default, which is cool.

All other UI modifications aside, I do still like rincewind's "easy-erase" function, which is where you hold down the pen after drawing a scribble, and when you lift the pen it's treated as an erase stroke. But I understand that this gets into some UI issues and you might not want to include it.

Matthijs wrote:
As for zoom and fullscreen: that is indeed an interesting one... I'm inclined to say that I'd like to at least zoom-to-fit fullscreen when the original one was zoom-to-fit.

This is interesting... Actually the zoom-to-fit question is the easiest problem. Unless the page is oddly shaped, a page that is zoomed-to-fit in toolbar mode will still be zoomed-to-fit in fullscreen mode with no change in zoom factor.

The tricky part is handling zoom when a user has selected a particular zoom box already. In that case, it might be possible to remember that zoom box to change the zoom for fullscreen. But this would require some work. Also, remember that if the zoom level changes from toolbar to fullscreen mode, then making that change will take longer than if we just toggle the toolbar, because the page must be re-rendered at the new zoom. I admit that it won't be quite as intuitive, however...
Back to top
View user's profile Send private message
Matthijs
iRex Employee
iRex Employee


Joined: 18 Jul 2006
Posts: 948
Location: Eindhoven

PostPosted: Tue Jul 10, 2007 3:41 pm    Post subject: Reply with quote

jharker wrote:
Unless the page is oddly shaped, a page that is zoomed-to-fit in toolbar mode will still be zoomed-to-fit in fullscreen mode with no change in zoom factor.

That doesn't sound right... If I have an A4 page zoomed to fit, going to fullscreen will definitely change the zoomfactor if it would have to remain zoom-to-fit. In case of a PDF with custom iLiad page-size you might be right, but for a generic PDF document, the tall size of the document will be the defining factor for zoom-to-fit.
Or am I missing something?
Back to top
View user's profile Send private message
jharker



Joined: 25 Apr 2007
Posts: 281
Location: Rochester, NY, USA

PostPosted: Tue Jul 10, 2007 10:37 pm    Post subject: Reply with quote

Oh- whoops, no, you're right. I forgot that A4 pages have a taller form-factor than the iLiad screen. Also Letter pages, I think. Yes, you're right, that would be a problem for many standard pages and should be fixed / changed.
Back to top
View user's profile Send private message
tribble



Joined: 04 Aug 2006
Posts: 645
Location: Bonn, Germany

PostPosted: Wed Jul 11, 2007 7:50 am    Post subject: Reply with quote

I would like to see, that if you are zoomed in, the toggling between fullscreen and normal should also fit the zoom area to the maximum in the available screenspace.
Back to top
View user's profile Send private message Visit poster's website
rincewind



Joined: 10 Jan 2007
Posts: 177
Location: Norway

PostPosted: Mon Jul 16, 2007 7:16 am    Post subject: Reply with quote

Matthijs wrote:

If deleting all scribbles on a page is to be added, it would need to use a toolbar icon. Maybe we can use the trashcan for that, as the Notes app does...


The "quick erase" is the "gesture" style erasing - scribble and hold the pen for a second and it will erase the line and anything the line crosses. The whole idea is to not use any icons or modes. If you solve sudokus or word puzzles, quick erase is positively required. Smile

/R
Back to top
View user's profile Send private message
jharker



Joined: 25 Apr 2007
Posts: 281
Location: Rochester, NY, USA

PostPosted: Sun Aug 19, 2007 5:13 pm    Post subject: Reply with quote

Hiya!

Well, as you may have guessed from my other posts, I'm back from my long hiatus and have a little time to spend on projects like the fullscreen patch. Over the next few days I'm going to try to develop a list of exact specifications for the patch's functionality. Then I'll sit down and code it, then debug it. Depending on what "real" work comes my way, I figure it'll take about a week, maybe two.

In the meantime, if anyone has suggestions or comments, now's the time to bring it up!

Matthijs, eventually we'll need that modified toolbar, or at least I'll need to know how you'll reference the new toolbar button in the software. In the meantime, I'll make an unused button (long down arrow?) do the job of the toolbar button, and when the new toolbar is ready I'll just move the code.
Back to top
View user's profile Send private message
kusmi



Joined: 03 Aug 2006
Posts: 136
Location: Zurich, Switzerland

PostPosted: Fri Feb 15, 2008 9:50 pm    Post subject: Reply with quote

I remember installing an pre-version of this one on my iLiad and it seemed usable, but have not seen it appearing in the mainstream release from iRex - is there still work to be done before integrating it into the "real" software release?
Back to top
View user's profile Send private message
Display posts from previous:   
Post new topic   Reply to topic    iRex Forum Forum Index -> iPDF Viewer All times are GMT + 1 Hour
Goto page 1, 2  Next
Page 1 of 2

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


You can find our complete guidelines here.
Powered by phpBB © 2001, 2005 phpBB Group