Feedback on 0.1.1 of the java area writer

[Tokugawa]

Some more feedback is below.

I have attached two screen shots.

I loaded an existing area into the area writer to see how it handled a large area file and then noticed the area header info below was not parsed correctly. "Area_Header.png" shows the first few letters of the area name being cut off, as well as the author name not being quite right.

"Edit_Mobs" shows there is no title in the title bar. The "Edit Objects", "Edit Current Area" and "Area Header" dialog also needs a title. I noticed the dialog for Edit Rooms does have a title, so that's good. Basically, any dialog that does not have a title should. Then it's apparent what you're doing at a glance.

The dialogs to edit Mobs, Objects and Mob resets need to be made wider so you do not have to scroll back and forth to see the whole name of the objects and mobs you are editing.

Would it be possible to make the input fields for room, mob, and object descriptions and long descriptions for mobs be a maximum of eighty characters wide? This would make it easier to format the rooms and descriptions when writing the zone. We want the width to be fixed for those input fields as telnet is eighty characters wide by default and we want to stick to that. If they are not, I can see formatting issues with descriptions as there would be no way to know how much room you have to write the descriptions.

A nice to have feature would also be to have a left and right arrow at the bottom of the rooms, mobs and objects edit dialog. So that once you had a room, mob or object in edit mode, you could click or use a hot-key to jump to the next or previous room, mob or object. This would aid in looking thru the rooms, mobs or objects without having to constantly click cancel to go back to the room, mob or object list, select the next one you want to look at, and then open it.

I also noticed that "unparsed.txt" was created, but there was nothing in it. Should there have been?

I think this is coming along great, thanks very much for doing this!

-Tokugawa

[Tokugawa]

Could we have the default reset count for Equip, Give and Place be 100? Also, it would be nice to have the Object and Mob default be 1. Then if needed the author could change the object or mob count if warranted.

Thanks!

[Thor]

I loaded an existing area into the area writer to see how it handled a large area file and then noticed the area header info below was not parsed correctly. "Area_Header.png" shows the first few letters of the area name being cut off, as well as the author name not being quite right.


I still can't reproduce this area. I copied are mystic wood line from the areas command online and put it in a test area file. I can't make it do what you're seeing. I attached a screen shot of mine. Can you send me the #AREA line you're using?



"Edit_Mobs" shows there is no title in the title bar. The "Edit Objects", "Edit Current Area" and "Area Header" dialog also needs a title. I noticed the dialog for Edit Rooms does have a title, so that's good. Basically, any dialog that does not have a title should. Then it's apparent what you're doing at a glance.


I've added more. I think I have them all, but I'll keep watching for this.



The dialogs to edit Mobs, Objects and Mob resets need to be made wider so you do not have to scroll back and forth to see the whole name of the objects and mobs you are editing.

Would it be possible to make the input fields for room, mob, and object descriptions and long descriptions for mobs be a maximum of eighty characters wide? This would make it easier to format the rooms and descriptions when writing the zone. We want the width to be fixed for those input fields as telnet is eighty characters wide by default and we want to stick to that. If they are not, I can see formatting issues with descriptions as there would be no way to know how much room you have to write the descriptions.


I'll be working on this as well.



A nice to have feature would also be to have a left and right arrow at the bottom of the rooms, mobs and objects edit dialog. So that once you had a room, mob or object in edit mode, you could click or use a hot-key to jump to the next or previous room, mob or object. This would aid in looking thru the rooms, mobs or objects without having to constantly click cancel to go back to the room, mob or object list, select the next one you want to look at, and then open it.

I can do this. I assume left is back one vnum and right is forward one vnum? Would you like to see if the next vnum doesn't exist? I could have it create a new room with the next available vnum.



I also noticed that "unparsed.txt" was created, but there was nothing in it. Should there have been?


I don't need this file anymore so I turned it off. It shouldn't be there in the next version.



I think this is coming along great, thanks very much for doing this!

-Tokugawa

Thanks to you also for testing so much.

Thor

P.S. I also updated the defaults as you requested in the other post. It will change with the next version.

[Tokugawa]

I still can't reproduce this area. I copied are mystic wood line from the areas command online and put it in a test area file. I can't make it do what you're seeing. I attached a screen shot of mine. Can you send me the #AREA line you're using?


Below is the first line from that area file.


#AREA 5 50 Kaern The Mystic Wood~ F 0




I can do this. I assume left is back one vnum and right is forward one vnum?
Correct


Would you like to see if the next vnum doesn't exist? I could have it create a new room with the next available vnum.
That would be handy, good idea. Can you also do short cut keys? Maybe ctrl+n for next and ctrl+p for previous.




I don't need this file anymore so I turned it off. It shouldn't be there in the next version.
Cool.




P.S. I also updated the defaults as you requested in the other post. It will change with the next version.

Awesome

[Tokugawa]

Would is be possible to only show the .are files for the Open dialog?

This would be nice so then only the files the editor supports are listed.

[Thor]

Would is be possible to only show the .are files for the Open dialog?

This would be nice so then only the files the editor supports are listed.

I agree this would be nice, but believe it or not, it's harder than I was hoping. It will get in there at some point. I have to extend the FileFilter class, etc...
I wish I could just say "Show only *.ARE"

Oh well.

Thor

[Tokugawa]

Would is be possible to only show the .are files for the Open dialog?

This would be nice so then only the files the editor supports are listed.

As an addition to this. Remember the last directory you opened previously.

[Tokugawa]

When you go to select a file to open, but then click the cancel button, the below exception is generated.



Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at java.io.FileInputStream.<init>(FileInputStream.java:133)
at java.io.FileReader.<init>(FileReader.java:72)
at tfcareawriter.Area.LoadArea(Area.java:276)
at tfcareawriter.MainView.btnLoadAreaActionPerformed( MainView.java:855)
at tfcareawriter.MainView.access$2400(MainView.java:3 0)
at tfcareawriter.MainView$27.actionPerformed(MainView .java:781)
at javax.swing.AbstractButton.fireActionPerformed(Abs tractButton.java:2012)
at javax.swing.AbstractButton$Handler.actionPerformed (AbstractButton.java:2335)
at javax.swing.DefaultButtonModel.fireActionPerformed (DefaultButtonModel.java:404)
at javax.swing.DefaultButtonModel.setPressed(DefaultB uttonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseRe leased(BasicButtonListener.java:253)
at java.awt.Component.processMouseEvent(Component.jav a:6108)
at javax.swing.JComponent.processMouseEvent(JComponen t.java:3276)
at java.awt.Component.processEvent(Component.java:587 3)
at java.awt.Container.processEvent(Container.java:210 5)
at java.awt.Component.dispatchEventImpl(Component.jav a:4469)
at java.awt.Container.dispatchEventImpl(Container.jav a:2163)
at java.awt.Component.dispatchEvent(Component.java:42 95)
at java.awt.LightweightDispatcher.retargetMouseEvent( Container.java:4461)
at java.awt.LightweightDispatcher.processMouseEvent(C ontainer.java:4125)
at java.awt.LightweightDispatcher.dispatchEvent(Conta iner.java:4055)
at java.awt.Container.dispatchEventImpl(Container.jav a:2149)
at java.awt.Window.dispatchEventImpl(Window.java:2478 )
at java.awt.Component.dispatchEvent(Component.java:42 95)
at java.awt.EventQueue.dispatchEvent(EventQueue.java: 604)
at java.awt.EventDispatchThread.pumpOneEventForFilter s(EventDispatchThread.java:275)
at java.awt.EventDispatchThread.pumpEventsForFilter(E ventDispatchThread.java:200)
at java.awt.EventDispatchThread.pumpEventsForHierarch y(EventDispatchThread.java:190)
at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:185)
at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:177)
at java.awt.EventDispatchThread.run(EventDispatchThre ad.java:138)

[Thor]

When you go to select a file to open, but then click the cancel button, the below exception is generated.



Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at java.io.FileInputStream.<init>(FileInputStream.java:133)
at java.io.FileReader.<init>(FileReader.java:72)
at tfcareawriter.Area.LoadArea(Area.java:276)
at tfcareawriter.MainView.btnLoadAreaActionPerformed( MainView.java:855)
at tfcareawriter.MainView.access$2400(MainView.java:3 0)
at tfcareawriter.MainView$27.actionPerformed(MainView .java:781)
at javax.swing.AbstractButton.fireActionPerformed(Abs tractButton.java:2012)
at javax.swing.AbstractButton$Handler.actionPerformed (AbstractButton.java:2335)
at javax.swing.DefaultButtonModel.fireActionPerformed (DefaultButtonModel.java:404)
at javax.swing.DefaultButtonModel.setPressed(DefaultB uttonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseRe leased(BasicButtonListener.java:253)
at java.awt.Component.processMouseEvent(Component.jav a:6108)
at javax.swing.JComponent.processMouseEvent(JComponen t.java:3276)
at java.awt.Component.processEvent(Component.java:587 3)
at java.awt.Container.processEvent(Container.java:210 5)
at java.awt.Component.dispatchEventImpl(Component.jav a:4469)
at java.awt.Container.dispatchEventImpl(Container.jav a:2163)
at java.awt.Component.dispatchEvent(Component.java:42 95)
at java.awt.LightweightDispatcher.retargetMouseEvent( Container.java:4461)
at java.awt.LightweightDispatcher.processMouseEvent(C ontainer.java:4125)
at java.awt.LightweightDispatcher.dispatchEvent(Conta iner.java:4055)
at java.awt.Container.dispatchEventImpl(Container.jav a:2149)
at java.awt.Window.dispatchEventImpl(Window.java:2478 )
at java.awt.Component.dispatchEvent(Component.java:42 95)
at java.awt.EventQueue.dispatchEvent(EventQueue.java: 604)
at java.awt.EventDispatchThread.pumpOneEventForFilter s(EventDispatchThread.java:275)
at java.awt.EventDispatchThread.pumpEventsForFilter(E ventDispatchThread.java:200)
at java.awt.EventDispatchThread.pumpEventsForHierarch y(EventDispatchThread.java:190)
at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:185)
at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:177)
at java.awt.EventDispatchThread.run(EventDispatchThre ad.java:138)


This has been fixed and will be available in version 0.1.3.

[Thor]

Here's the changelog:


Version 0.1.3
NEW FEATURES
* Added door resets
* Added mob specfuns section in specials
* Added allow door triggers
* Added prevent door triggers
* Added rspecs
* The {}s are now optional around level range in the area header. The area file will save without them.
BUG FIXES
* Fixed a bug where if you hit cancel after open area file an exception was thrown.
* Fixed a bug where if you didn't select a key in an exit an exception was thrown.


URL:
http://kirbyprogramming.railsplayground.net/tfc/TFC_Area_Writer-0.1.3.zip

[Thor]

I agree this would be nice, but believe it or not, it's harder than I was hoping. It will get in there at some point. I have to extend the FileFilter class, etc...
I wish I could just say "Show only *.ARE"

Oh well.

Thor

This has been done. It will be available in version 0.1.4

[Thor]

As an addition to this. Remember the last directory you opened previously.

This has been done and will be available in version 0.1.4

[Tokugawa]

Woo Hoo!

You're just moving right along with this project.

Good work!

-Tokugawa

[Thor]

Version 0.1.4 is now here!

This version should be able to do everything in an area file now. I lack two more things for the release of version 1.0.0.

1) Walkthrough Mode
2) Save to the same area file that was loaded or Save As...

After 1.0.0 I'll start adding the extra requested features like keyboard shortcuts, forward and back buttons, etc....

URL:
http://kirbyprogramming.railsplayground.net/tfc/TFC_Area_Writer-0.1.4.zip

Here are the details for version 0.1.4:



Version 0.1.4
NEW FEATURES
* Added shops
* The file open dialog now only shows *.are files
BUG FIXES
* No bugs reported for this version