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

Xml wizard #172

Merged
merged 70 commits into from
Jun 18, 2015
Merged

Xml wizard #172

merged 70 commits into from
Jun 18, 2015

Conversation

pjvanwijk
Copy link
Collaborator

met de wizard kan je nu ook bestaande xml files editen

@pjvanwijk
Copy link
Collaborator Author

... zucht. testje vergeten

@pjvanwijk pjvanwijk closed this Jun 10, 2015
…ics-3 into xml-wizard

Fixed some merge conflicts because of added method to dialog. Merge to merge with master

Conflicts:
	src/main/java/model/input/file/PlainTextFile.java
	src/main/java/view/Dialog.java
@pjvanwijk pjvanwijk reopened this Jun 10, 2015
@pjvanwijk pjvanwijk closed this Jun 10, 2015
@pjvanwijk pjvanwijk reopened this Jun 11, 2015
@pjvanwijk pjvanwijk reopened this Jun 17, 2015
@JensLangerak
Copy link
Collaborator

date time format field is in het begin niet gedisabled, terwijl string is geselecteerd

@JensLangerak
Copy link
Collaborator

geeft toelichting bij het format, bijvoorbeeld een ? met als je erover hovert toelichting

@JensLangerak
Copy link
Collaborator

filenamecolumn daar wordt format nooit gedisabled

@JensLangerak
Copy link
Collaborator

plain textfiled specification en add filename column worden gereset als je een andere file aanklikt

@JensLangerak
Copy link
Collaborator

bij excel file kan ik slechts een column toevoegen

@pjvanwijk pjvanwijk reopened this Jun 18, 2015
@pjvanwijk
Copy link
Collaborator Author

Is het beter dat ik dingen zoals help tooltips in een nieuwe pull request doe? deze is al reusachtig

@Boudewijn26
Copy link
Collaborator

Code ziet er prima uit en volgens mij werkt het ook, als de scanner wel closed mag hij van mij.

louisgosschalk added a commit that referenced this pull request Jun 18, 2015
@louisgosschalk louisgosschalk merged commit 5db9ac4 into master Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants