search

ucsb-cs56-projects / cs56-misc-map-gui

-
0 stars 7 forks source link

Issue: Zoom In Function #12

Open thebohster opened 11 years ago

thebohster commented 11 years ago

@spencerpao and I would like to work on this issue. ~moderator: @CARammelkamp

pconrad commented 11 years ago

So, this project is currently assigned to another team, namely @iamnearl and @jdmoody. I think it will be too complicated to have you working in the same code unless you can figure out a way to stay out of each others way.

Also, adding a zoom function would be a VERY BAD IDEA unless and until the multiple methods (one per building) are refactored into a single "displayBuilding" method (see issue #13 ). And I've assigned them that issue, and there isn't really time to have them finish that then hand it off to you.

As an alternative, I'm ok with you taking a subset of the code here and working on a "zoom in" / "zoom out" tutorial---where you start with the code from this project, and strip it down to the BARE essential-----namely hard code it so that it displays ONLY the panel that comes up when you click on the HFH button in the original---then add zoom in/zoom out capabilities to THAT. This would be in a separate project called cs56-tutorials-image-zoom. I'm going to set up an empty repo with that in it now along with the issue. The issue will be worth 300 if you make a NICE repo with ONLY the minimum code, with a good build.xml, javadoc, etc.---and write it in a way that a future student can easily incorporate it into the map project at a later stage.

pconrad commented 11 years ago

I've assigned this to Bohan and Spencer---but you may ONLY do this one if you do these first:

https://github.com/UCSB-CS56-Projects/cs56-tutorials-image-zoom/issues/1 https://github.com/UCSB-CS56-Projects/cs56-tutorials-image-zoom/issues/2

and then work with @iamnearl and @jdmoody to integrate the code with theirs.

Otherwise, you should just concentrate on the two issues above.