Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: Save comics to SD card #15

Open
tcoxon opened this issue Jan 28, 2010 · 8 comments
Open

Enhancement: Save comics to SD card #15

tcoxon opened this issue Jan 28, 2010 · 8 comments

Comments

@tcoxon
Copy link
Owner

tcoxon commented Jan 28, 2010

One user requested this in comments.

@damccull
Copy link
Contributor

damccull commented Apr 1, 2011

Seems like you already have code in place to do this (share image). Adding a menu option to call that function shouldn't be too hard.

@tcoxon
Copy link
Owner Author

tcoxon commented Apr 1, 2011

Yes, it's not much work to implement it. What's been putting me off is what the behaviour should be. Should it ask the user for a location to save it, or should it save to a predictable location (e.g. /sdcard/xkcdViewer/123.png) and use toast to display it? Need to check what other apps do.

@damccull
Copy link
Contributor

damccull commented Apr 1, 2011

I'm not sure I'd blindly follow other apps practices. Most of them like to
clutter users sdcards by creating all their new folders on the root instead
ofof using appropriate folders that are likely already there.

For behavior, I suggest adding a preference to your settings activity that
allows the user to set the location to save images, and have a default value
of /sdcard/Pictures.

http://developer.android.com/guide/topics/data/data-storage.html#filesExternal
The above link contains suggested file storage practices, and though I'm
sure youve already seen it, I figured I'd point it out since most apps
ignore these practices and place their caches and export directories
directly on the sdcard root.

@horkana
Copy link

horkana commented Jun 28, 2011

Like the viewer. Would love to have things cached for offline reading.
Hope you can support mouseover comments in the offline version too.

The reasonable default would probably be
/sdcard/Pictures/xkcd
or possibly
/sdcard/Pictures/Comics/xkcd

@MatrixFrog
Copy link
Contributor

@horkana just in case you weren't aware, you can get the mouseover text by clicking on the image itself. I have a touchscreen so having it on hover obviously doesn't work on my phone -- not sure if there are other devices where it would make sense.

@horkana
Copy link

horkana commented Jun 28, 2011

Saw from the documentation that mouseover text was supported and after a while guessed that I just needed to click to support it.
I'm not sure how else you might have implemented it but my first guess was that you might just show the text beneath the image but click to show works fine. I suppose maybe some kind of on screen overlay might make it a little more discoverable but as I said I guessed soon enough. (My tiny screen makes me concerned about the accessibility of presenting text as an image is a whole other issue the downside of the simple image formats used for most comics but unless Randall Munroe decides to present the text in an accessible way that is far beyond the scope of what I'd expect.)

@nognir
Copy link

nognir commented Jun 25, 2013

It's kind of disappointing to see that this viewer would the best, save for that missing feature... I found the in-app downloads in the app from Alex Avance (which I'm using) to be the most convenient. The comic is saved offline, accessed in-app from the catalogue and viewed just like the ones still online.

@tcoxon
Copy link
Owner Author

tcoxon commented Jul 19, 2013

Sorry, I haven't been giving this project the attention it deserves. I have more commitments now than I did 3 years ago, and it's hard to find the time to work on this.

I'll still be fixing bugs and accepting pull requests, but I don't plan to implement any new features myself. If you find Alex Avance's app to work better for your needs then by all means go ahead and use it. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants