Opened 13 years ago

Closed 12 years ago

Last modified 2 years ago

#8840 closed patch

Zoom options on Nintendo DS

Reported by: SF/xanathar Owned by: agent-q
Priority: normal Component: Port: NDS
Keywords: Cc:
Game:

Description

I'm submitting this patch (against scummvm-0.11.1 sources) which does the following :

- Remove the "Fixed 200% zoom on zoomed screen" checkbox + Add three radio buttons named "dynamic zoom" (which is the classic zoom in nds scummvm), "Zoom 200%" (which is the same as the previous checkbox) and "Zoom 100%" (which is new and is a fixed zoom as 100% size - more comfortable in many situations)

The option is implemented as an integer setting.

I don't know if 100% zoom is just my personal preference or if it's agreed by many, I'm submitting it anyway in case anyone is interested in integration.

I've build binaries with it applied (you can find them at http://www.mastropaolo.com/2008/03/13/patch-for-scummvm-on-nds/)

Thank you very much, Marco

Ticket imported from: #1913689. Ticket imported from: patches/945.

Attachments (1)

dsscalepatch.diff (6.5 KB ) - added by SF/xanathar 13 years ago.
Patch for files under backends/platform/ds/arm9/source/

Download all attachments as: .zip

Change History (10)

by SF/xanathar, 13 years ago

Attachment: dsscalepatch.diff added

Patch for files under backends/platform/ds/arm9/source/

comment:1 by fingolfin, 13 years ago

Owner: set to agent-q

comment:2 by agent-q, 13 years ago

Hi,

Thanks for your patch, it looks great at first glance. I will integrate it next time I look at ScummVM.

100% zoom is already available, as you probably know, using the dynamic zoom and zooming out, it will snap at 100% and show a '1x' icon on the screen, but I think your way is more obvious, and will save to the options file too.

Thanks again,

- Neil

comment:3 by sev-, 12 years ago

What is the status of this item?

comment:4 by agent-q, 12 years ago

I'm planning to integrate this, in some form, in the next few days.

comment:5 by fingolfin, 12 years ago

What is the status of this item?

comment:6 by fingolfin, 12 years ago

So, any news? Has this been integrated? It's been more than half a year since the last update :-/.

comment:7 by agent-q, 12 years ago

I did implement this feature last year. It went in for 0.12.0, but I didn't use much of this patch to do it.

Still, I did use some.

I'm closing this patch now.

comment:8 by agent-q, 12 years ago

Status: newclosed

comment:9 by digitall, 2 years ago

Component: Port: NDS
Note: See TracTickets for help on using tickets.