On 18/11/2017 00:01, Lubomir I. Ivanov wrote:

*@neolit123* requested changes on this pull request.

@willemferguson <https://github.com/willemferguson>

hi, Willem. i've made a more detailed review for this commit. it needs work, before we can accept it in. so, please, comment on my comments if you'd like.

most importantly, please do not change the bundled "One Dive.html|file and instead commit a new one - e.g.|One Dive .html`.

you might have to learn how to |git rebase|.

Lubomir,

I think I will just send a patch that does not include the automatic text sizing in the notes text field. This will remove the hacked code. I will work with that separately and if I can get a more elegant solution submit that as a separate PR.

I agree that we should keep the old version.

Does this sound acceptable?

Kind regards,
willem


--
This message and attachments are subject to a disclaimer.
Please refer to http://upnet.up.ac.za/services/it/documentation/docs/004167.pdf for full details.
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to