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
Added popup Syntax Selector #8
Conversation
…s, i.e. iPhone X. Note: On iPhone X, if you exit app and then press on app icon to enter it again, the background scrolling does not start again. Just static image. Needs investigating further. Moved Done button on the Previous Pastes view to the right in order to conform to all other subviews.
…x. Currently is based on the pastebin.com supported languages, not Highlightr, so some syntax will not work. Next step is to merge the Pastebin.com supported languages and the Highlightr supported languages with the Pastebin list being the master and if not available then it will reset to default or no syntax at all.
…match the Highlightr syntaxes and will highlight the code accordingly. If no match, then it will default where Highlightr will automatically choose the closest matching highlighter. Choosing None will turn off highlighter.
Looks great! Thanks! I’m open to suggestions on how to fix the scrolling background. We could either try something completely new and different, or try to fix it everywhere. Did you have anything in mind? |
Merge pull request jonluca#8 from Aecasorg/master
I think I fixed the issue that you had or at least the one you mentioned on GitHub.
What phone are you using when you test? What issues do you see currently with the background?
I’ve got an iPhone SE so I’ll test with the actual device and see if there are any issues.
I’d prefer to stick with scrolling background you set up as I quite like that, so I’ll see if I can fix it but I need to know exactly what issues you see so I hopefully can replicate them here.
Henrik
… On 28 Mar 2018, at 09:54, JonLuca De Caro ***@***.***> wrote:
Looks great! Thanks!
I’m open to suggestions on how to fix the scrolling background. We could either try something completely new and different, or try to fix it everywhere. Did you have anything in mind?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5Y9D1Bi5s-JKnRi_S3sfqGrcOlDKEks5ti0G7gaJpZM4S-H5f>.
|
Ok, I can’t test it on my phone since I’m not in the “development team”. No worries. Maybe you can do the actual device testing?
Cheers!
… On 28 Mar 2018, at 09:54, JonLuca De Caro ***@***.***> wrote:
Looks great! Thanks!
I’m open to suggestions on how to fix the scrolling background. We could either try something completely new and different, or try to fix it everywhere. Did you have anything in mind?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5Y9D1Bi5s-JKnRi_S3sfqGrcOlDKEks5ti0G7gaJpZM4S-H5f>.
|
Looks good to me! Just tested it, it all works. Also, can you not test it in the simulator? You should be able to build locally even without an Apple Dev account. I also went through and reformatted some stuff just slightly, to maintain a more consistent style. You can see the commit here Thanks again! |
You’re welcome!
Yes, I always test in the simulators. I usually test with iPhone SE, 8 and X simulators in order to cover most variations. Simulators are not perfect so there are still issues that can come up when you use the app on a real device, which is now what has happened it seems. All the simulations worked however when I now downloaded the app from the App Store the Syntax Highlighter does not work on my iPhone SE. I’m not sure now how to solve this since I can’t test on my device. I’ll download your newest commit and see it I can replicate the issue.
Nice! It is always good to clean up code now and then :)
Oh, one thing. Could I ask a favour from you at some point in future? Very simple, but it could help me when I look for work later this year.
Henrik
… On 28 Mar 2018, at 18:07, JonLuca De Caro ***@***.***> wrote:
Looks good to me! Just tested it, it all works.
Also, can you not test it in the simulator? You should be able to build locally even without an Apple Dev account.
I also went through and reformatted some stuff just slightly, to maintain a more consistent style. You can see the commit here <175ce6f>
Thanks again!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5YxkSR_UAaDOd4AF2E-N_tolRA4JLks5ti7U2gaJpZM4S-H5f>.
|
One more thing…
Do you know anyone with an iPhone X? In the simulator I get the error "Bad API request, invalid api_paste_format” when trying to send a paste to pastebin.com but again, simulators don’t always correspond entirely to real devices so it could be a non-issue. My iPhone X simulator behaves different from the other sims so I don’t know if there’s difference there. Technically there should not be a difference since they run the same OS. Is there anyone you know that could test this for us?
On an separate note I have been thinking. I like your quick paste function on the main screen on the app and I would like to extend it by using 3DTouch so you can get an option to do it from the home screen icon instead of entering the app and then do it. How does that sound?
Henrik
… On 29 Mar 2018, at 07:57, L. Henrik Gustavii ***@***.***> wrote:
You’re welcome!
Yes, I always test in the simulators. I usually test with iPhone SE, 8 and X simulators in order to cover most variations. Simulators are not perfect so there are still issues that can come up when you use the app on a real device, which is now what has happened it seems. All the simulations worked however when I now downloaded the app from the App Store the Syntax Highlighter does not work on my iPhone SE. I’m not sure now how to solve this since I can’t test on my device. I’ll download your newest commit and see it I can replicate the issue.
Nice! It is always good to clean up code now and then :)
Oh, one thing. Could I ask a favour from you at some point in future? Very simple, but it could help me when I look for work later this year.
Henrik
> On 28 Mar 2018, at 18:07, JonLuca De Caro ***@***.*** ***@***.***>> wrote:
>
> Looks good to me! Just tested it, it all works.
>
> Also, can you not test it in the simulator? You should be able to build locally even without an Apple Dev account.
>
> I also went through and reformatted some stuff just slightly, to maintain a more consistent style. You can see the commit here <175ce6f>
> Thanks again!
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5YxkSR_UAaDOd4AF2E-N_tolRA4JLks5ti7U2gaJpZM4S-H5f>.
>
|
OK, I’m sorry. I did not realise you had already implemented 3D Touch. Great! So just ignore my suggestion… :)
Henrik
… On 29 Mar 2018, at 08:55, L. Henrik Gustavii ***@***.***> wrote:
One more thing…
Do you know anyone with an iPhone X? In the simulator I get the error "Bad API request, invalid api_paste_format” when trying to send a paste to pastebin.com <http://pastebin.com/> but again, simulators don’t always correspond entirely to real devices so it could be a non-issue. My iPhone X simulator behaves different from the other sims so I don’t know if there’s difference there. Technically there should not be a difference since they run the same OS. Is there anyone you know that could test this for us?
On an separate note I have been thinking. I like your quick paste function on the main screen on the app and I would like to extend it by using 3DTouch so you can get an option to do it from the home screen icon instead of entering the app and then do it. How does that sound?
Henrik
> On 29 Mar 2018, at 07:57, L. Henrik Gustavii ***@***.*** ***@***.***>> wrote:
>
> You’re welcome!
>
> Yes, I always test in the simulators. I usually test with iPhone SE, 8 and X simulators in order to cover most variations. Simulators are not perfect so there are still issues that can come up when you use the app on a real device, which is now what has happened it seems. All the simulations worked however when I now downloaded the app from the App Store the Syntax Highlighter does not work on my iPhone SE. I’m not sure now how to solve this since I can’t test on my device. I’ll download your newest commit and see it I can replicate the issue.
>
> Nice! It is always good to clean up code now and then :)
>
> Oh, one thing. Could I ask a favour from you at some point in future? Very simple, but it could help me when I look for work later this year.
>
> Henrik
>
>
>> On 28 Mar 2018, at 18:07, JonLuca De Caro ***@***.*** ***@***.***>> wrote:
>>
>> Looks good to me! Just tested it, it all works.
>>
>> Also, can you not test it in the simulator? You should be able to build locally even without an Apple Dev account.
>>
>> I also went through and reformatted some stuff just slightly, to maintain a more consistent style. You can see the commit here <175ce6f>
>> Thanks again!
>>
>> —
>> You are receiving this because you authored the thread.
>> Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5YxkSR_UAaDOd4AF2E-N_tolRA4JLks5ti7U2gaJpZM4S-H5f>.
>>
>
|
Yes, I have the iPhone X and I'm running into that issue as well. I'm looking into it. I've also been getting some random crashes with typing in user and switching syntaxes, not really sure what it is. I'm pretty busy this week but I'll try to take a look soon. And sure! What favor did you need? |
Hmm odd the error isn't happening for me anymore. Not sure what's up. |
Hmm… Keep trying it out and let me know what errors you get (if you get any), so I can investigate. Thanks!
Favour? Well, it is easy. If you happen to have a LinkedIn account, if you could endorse some of my relevant skills it would help greatly. It is not urgent though.
Henrik
… On 29 Mar 2018, at 17:34, JonLuca De Caro ***@***.***> wrote:
Yes, I have the iPhone X and I'm running into that issue as well. I'm looking into it. I've also been getting some random crashes with typing in user and switching syntaxes, not really sure what it is. I'm pretty busy this week but I'll try to take a look soon.
And sure! What favor did you need?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ATB5Y5QCWAp4FCTaFrRAFYo9ZRmVEbv0ks5tjP8egaJpZM4S-H5f>.
|
In pasteview, a syntax selector picker has been added. It defaults to language set in options menu. If there's not a matching language that Hightlightr supports then it is set to let Highlightr automatically choose.
Partially fixed the background bug. On an iPhone SE the background image starts lower so you see top edge, however, I don't actually know if this will actually be like this on an actual phone. Not tested.
Changed the order around for main VC buttons. More logical perhaps but looks slightly worse on an iPhone SE. Should we keep or change back?
Let me know if you want any changes to the above.
Enjoy!
Henrik