OpenPonk / openponk

The OpenPonk modeling platform
https://openponk.github.io/
MIT License
43 stars 9 forks source link

Added code for displaying project location and for automatic update when saving (related to #80) #88

Closed sdasda7777 closed 1 year ago

sdasda7777 commented 1 year ago

(I don't know why there's so many changes, I only modified two or so lines, and the other files seem unchanged anyways)

Fixes #80 🤦‍♂️🤦‍♂️🤦‍♂️

sdasda7777 commented 1 year ago

New project still looks like this:

image

But after saving, or when opening previously saved file, the path is shown as well:

image

JanBliznicenko commented 1 year ago

Thank you. The diffs for entire files seem like differences in line endings (or they usually are). I believe most files originally had CRLFs, but I wonder why only these were changed, to what line endings and why those with real changes are unaffected.

sdasda7777 commented 1 year ago

I see, definitely could be the case. Should I rebase it and leave out the "unmodified" files, then?

JanBliznicenko commented 1 year ago

Yes, please do. Before you do, try to find what were the original and new line endings, please, so we can find out what can we do about that in the future (I fight with line endings sometimes as well).

sdasda7777 commented 1 year ago

As far as I can tell, it seems the files I modified already had Windows newlines, whereas the other ones had Unix newlines originally 🤔 Either way it's disappointing to see that Iceberg commited something that wasn't shown on the commit review screen at all 😢

sdasda7777 commented 1 year ago

Either way it's disappointing to see that Iceberg commited something that wasn't shown on the commit review screen at all 😢

Although, could it be possible one of the git hooks did it? I think not, since it would most likely require the Iceberg to commit changes to those files in the first place, but I'm not completely sure.

On topic of git, I think git has some setting for forcing a newline kind for a filetype: that might be a good idea to use.

sdasda7777 commented 1 year ago

(It seems you have to link issue from the PR, not the other way around: #80 )

Fixes #80 🤦‍♂️