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.

Sign up for GitHub

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

Jump to bottom

Jdk10 fix: #1223 #1212 #1209 #1665 #1237

Open
wants to merge 22 commits into
base: jdk10
Choose a base branch
from
Open

Jdk10 fix: #1223 #1212 #1209 #1665 #1237

wants to merge 22 commits into from

Conversation

SchweinchenFuntik
Copy link
Contributor

No description provided.

Thijs Koppen and others added 19 commits March 14, 2020 18:29
🚸 Change MapConversionListener's sourceToTarget reference map to use …
c749edf 
…the source's key as key instead of the recreated entry
@edvin
Merge pull request edvin#1183 from Thijsiez/improve-mapconversionlist…
4309331 
…ener

Improving MapConversionListener
@SchweinchenFuntik
fix edvin#1191
d300dd7
@edvin
Merge pull request edvin#1195 from SchweinchenFuntik/master
9934ffb 
fix edvin#1191 master
@Camork
fix json.float
4da1041
@edvin
Merge pull request edvin#1218 from Camork/fix-float
17cebe6 
Fix stack overflow when get float value on json
@HoldYourWaffle
Update dependency version in README
30ae80f
@ctadlock
Merge pull request edvin#1225 from HoldYourWaffle/patch-3
eea015c 
Update dependency version in README
@SchweinchenFuntik
Merge branches 'jdk10' and 'master' of https://github.com/edvin/torna…
60239ae 
…dofx into jdk10
@SchweinchenFuntik
fix: edvin#1223
871f753
@SchweinchenFuntik
fix: edvin#1212 edvin#1209
195793c
@SchweinchenFuntik
up version kotlin and indicated explicit type
9781649
@SchweinchenFuntik
Downgrade FX. Motivation:
a51a1df 
The JavaFX ecosystem is not yet ready for new versions. Also JavaFX 11 is LTS.
JavaFX 13 main problem - closing reflect calls. User can always upgrade JavaFX
@SchweinchenFuntik
fix: edvin#1165
63b5aad
@SchweinchenFuntik
doesn’t compile, I had to fix it.
f8c6842
@SchweinchenFuntik
doesn’t compile, I had to fix it.
fce9cfb
@SchweinchenFuntik
Merge branch 'jdk10' of https://github.com/SchweinchenFuntik/tornadofx …
06995d1 
…into jdk10
@SchweinchenFuntik
append locale: uk_UA, ru_UA
7f27830
@SchweinchenFuntik
fix: edvin#1236
3646954
@SchweinchenFuntik
Copy link
Contributor Author

@edvin @ctadlock are there any comments on these commits?

@edvin
Copy link
Owner

edvin commented May 27, 2020

@ctadlock This should go into the tornadofx2 repo now, shouldn't it?

@SchweinchenFuntik
Copy link
Contributor Author

*up javafx version (13)

@SchweinchenFuntik
Copy link
Contributor Author

SchweinchenFuntik commented Sep 1, 2020 •
edited

@edvin I made a serious mistake last merging with the main branch. Please take it and put it on the bintray. And then @ctadlock while with the repository TornadoFX 2 is not in a hurry.

this PR will replace #1219

PS: did not know in which file to add the canOrSetAccessMethod method, correct me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@SchweinchenFuntik @edvin @Camork @HoldYourWaffle @ctadlock