Githawk: ์ƒˆ ์˜ค๋ฅ˜ ๊ฒฝ๊ณ 

์— ๋งŒ๋“  2017๋…„ 09์›” 17์ผ  ยท  22์ฝ”๋ฉ˜ํŠธ  ยท  ์ถœ์ฒ˜: GitHawkApp/GitHawk

iPhone X๋Š” ์ƒํƒœ ํ‘œ์‹œ์ค„ ๊ฒฝ๊ณ ๋ฅผ ๊ตฌ์‹์œผ๋กœ ๋งŒ๋“ญ๋‹ˆ๋‹ค. ์ƒˆ๋กœ์šด ๊ฒƒ์„ ์ฐพ์•„์•ผ ํ•ฉ๋‹ˆ๋‹ค. ์–ด๋–ค ์•„์ด๋””์–ด?

๐Ÿ› bug

๊ฐ€์žฅ ์œ ์šฉํ•œ ๋Œ“๊ธ€

๋‚ด๊ฐ€ ์ˆ˜ํ–‰ํ•œ ์ž‘์—…์— ๋Œ€ํ•œ ๋น ๋ฅธ ๋ฏธ๋ฆฌ๋ณด๊ธฐ:

wgde5803

ํŠน์ง•:

  • ์™„์ „ํžˆ ๊ตฌ์„ฑ ๊ฐ€๋Šฅ(ํ…์ŠคํŠธ, ๊ธ€๊ผด, ์ƒ‰์ƒ, ๋ ˆ์ด์•„์›ƒ ๊ฐ„๊ฒฉ)
  • ์ธํ„ฐ๋ ‰ํ‹ฐ๋ธŒ

    • ์œ„๋กœ ์Šค์™€์ดํ”„ํ•  ๋•Œ ๊ณ ๋ฌด์ค„ ํšจ๊ณผ

    • ๋‹ซ์œผ๋ ค๋ฉด ์•„๋ž˜๋กœ ์Šค์™€์ดํ”„ํ•˜์„ธ์š”.

ํ•  ๊ฒƒ:

  • ์ง€์† ์‹œ๊ฐ„ ํ›„ ์ž๋™ ํ•ด์ œ
  • ๋ฒ„ํŠผ์„ ํƒญํ•  ๋•Œ ์œ„์ž„
  • ์—ฌ๋Ÿฌ ์ค„ ๋ ˆ์ด๋ธ” ์ˆ˜์ •
  • ๋ฌผ๋ฆฌํ•™ ์กฐ์ •

UIDynamics ๋ฐ Auto Layout w/ ์ด๊ฒƒ์— ๋Œ€ํ•ด ๊ฝค ๋ฐฐ์› ์Šต๋‹ˆ๋‹ค. ์ฆ๊ฑฐ์› ์–ด!

๋ชจ๋“  22 ๋Œ“๊ธ€

์•Œ๋ฆผ ์Šคํƒ€์ผ์€ ๊ฐ™์ง€๋งŒ ํƒญ ํ‘œ์‹œ์ค„์ด๋‚˜ ์ปค๋ฒ„ ํƒ์ƒ‰ ํ‘œ์‹œ์ค„ ์œ„์— ์•Œ๋ฆผ์ด ํ‘œ์‹œ๋ฉ๋‹ˆ๊นŒ? ๋˜๋Š” ์‚ฌ๋ก€๋ณ„๋กœ ๊ธฐ๋ณธ์ ์œผ๋กœ ํ•ด๊ฒฐํ•˜๊ณ  ๋” ๋งŽ์€ ์ƒํ™ฉ์— ๋งž๋Š” ์˜ค๋ฅ˜ ๋ฉ”์‹œ์ง€๋ฅผ ์ œ๊ณตํ•˜๋ ค๊ณ  ํ•ฉ๋‹ˆ๋‹ค.

ํ† ์ŠคํŠธ ์•„์ด๋””์–ด๊ฐ€ ์ •๋ง ์ข‹์•„์š”. ์‚ฌ๋ƒฅํ•˜๋Ÿฌ ๊ฐ€์ž!

GitHawk ์™€ ํ•จ๊ป˜

https://github.com/devxoul/Toaster - ๋งค์šฐ ๋น ๋ฅธ ๊ฒ€์ƒ‰์ด ์ด๋ฅผ ๋ณด์—ฌ์ฃผ๊ณ , ๋ชจ์–‘ ์žฌ์ •์˜๋ฅผ ์ œ๊ณตํ•˜๋ฏ€๋กœ ๋ช‡ ๊ฐ€์ง€ ํŒฉํ† ๋ฆฌ ๋ฉ”์„œ๋“œ๋ฅผ ์‚ฌ์šฉํ•˜์—ฌ ์˜ค๋ฅ˜/๊ธฐํƒ€ ๋ฉ”์‹œ์ง€๋ฅผ ๊ตฌ๋ณ„ํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๊นŒ?

https://github.com/devxoul/Toaster ๋ฅผ ๊ตฌํ˜„ํ•˜๋Š” pull request #483์„ ๋งŒ๋“ค์—ˆ์Šต๋‹ˆ๋‹ค.

์ž˜์ƒ๊ฒผ์–ด, ์ข€ ๋” ์‚ฌ์†Œํ•œ ๋ ˆ์ด์•„์›ƒ ๋ณ€๊ฒฝ

@rnystrom ์ด #483์—์„œ ์–ธ๊ธ‰ํ–ˆ๋“ฏ์ด ์šฐ๋ฆฌ๋Š” ์šฐ๋ฆฌ ์ž์‹ ์„ ๋งŒ๋“ค์–ด์•ผ ํ•ฉ๋‹ˆ๋‹ค. ๐Ÿ˜„

์ง‘์œผ๋กœ ๊ฐ€๋Š” ๋น„ํ–‰๊ธฐ์—์„œ ์ด ์ž‘์—…์„ ์‹œ์ž‘ํ–ˆ์Šต๋‹ˆ๋‹ค. ๊ณง ๋ญ”๊ฐ€๋ฅผ ๊ฐ–๊ฒŒ ๋  ๊ฒƒ์ž…๋‹ˆ๋‹ค! ์žฌ๋ฏธ์žˆ๋Š” ์•„์ด๋””์–ด๊ฐ€ ๋– ์˜ฌ๋ž์Šต๋‹ˆ๋‹ค. ๋ณ„๋„์˜ ํฌ๋“œ๊ฐ€ ๋  ์ˆ˜ ์žˆ๋„๋ก ๊ตฌ์ถ•ํ•ฉ๋‹ˆ๋‹ค. @BasThomas ์˜ ์•„์ด๋””์–ด๋ฅผ ์‚ฌ๋ž‘ํ•ฉ๋‹ˆ๋‹ค.

๋‚ด๊ฐ€ ์ˆ˜ํ–‰ํ•œ ์ž‘์—…์— ๋Œ€ํ•œ ๋น ๋ฅธ ๋ฏธ๋ฆฌ๋ณด๊ธฐ:

wgde5803

ํŠน์ง•:

  • ์™„์ „ํžˆ ๊ตฌ์„ฑ ๊ฐ€๋Šฅ(ํ…์ŠคํŠธ, ๊ธ€๊ผด, ์ƒ‰์ƒ, ๋ ˆ์ด์•„์›ƒ ๊ฐ„๊ฒฉ)
  • ์ธํ„ฐ๋ ‰ํ‹ฐ๋ธŒ

    • ์œ„๋กœ ์Šค์™€์ดํ”„ํ•  ๋•Œ ๊ณ ๋ฌด์ค„ ํšจ๊ณผ

    • ๋‹ซ์œผ๋ ค๋ฉด ์•„๋ž˜๋กœ ์Šค์™€์ดํ”„ํ•˜์„ธ์š”.

ํ•  ๊ฒƒ:

  • ์ง€์† ์‹œ๊ฐ„ ํ›„ ์ž๋™ ํ•ด์ œ
  • ๋ฒ„ํŠผ์„ ํƒญํ•  ๋•Œ ์œ„์ž„
  • ์—ฌ๋Ÿฌ ์ค„ ๋ ˆ์ด๋ธ” ์ˆ˜์ •
  • ๋ฌผ๋ฆฌํ•™ ์กฐ์ •

UIDynamics ๋ฐ Auto Layout w/ ์ด๊ฒƒ์— ๋Œ€ํ•ด ๊ฝค ๋ฐฐ์› ์Šต๋‹ˆ๋‹ค. ์ฆ๊ฑฐ์› ์–ด!

์˜ค ์™€์šฐ, ๋ฉ‹์ ธ ๋ณด์ธ๋‹ค! ์–ด๋–ป๊ฒŒ ๊ตฌํ˜„ํ–ˆ๋Š”์ง€ ๊ธฐ๋Œ€ํ•ด์ฃผ์„ธ์š” ๐Ÿ˜€

GitHawk ์™€ ํ•จ๊ป˜

๊ทธ ์ด๋ฏธ์ง€๋Š” ์•ฑ์˜ ๊ด€์ ์—์„œ ๋งค์šฐ ํ˜ผ๋ž€ ์Šค๋Ÿฌ์› ์Šต๋‹ˆ๋‹ค. ๊ทธ๋ƒฅ ์ด๋ฏธ์ง€์ฒ˜๋Ÿผ ๋ณด์ž…๋‹ˆ๋‹ค ๐Ÿ˜…

๊ทธ๋ž˜๋„ ์ข‹์•„ ๋ณด์ธ๋‹ค! (GitHub์—์„œ ๋ณผ ๋•Œ)

๋„ค, "์‹คํ–‰ ์ค‘์ธ ์ž‘์—…์— ๋Œ€ํ•œ ๋น ๋ฅธ ๋ฏธ๋ฆฌ๋ณด๊ธฐ:"๊ฐ€ ํ™•์‹คํžˆ ๋„์›€์ด ๋˜์—ˆ์Šต๋‹ˆ๋‹ค. gif ์ง€์› ๋ถ€ํƒ๋“œ๋ฆฝ๋‹ˆ๋‹ค! ๐Ÿ˜

NY ์‚ฌ์ง„ ๋ทฐ์–ด๊ฐ€ GIF๋ฅผ ์ง€์›ํ•˜๊ธฐ ๋•Œ๋ฌธ์— ์ด์ƒํ•˜์ง€๋งŒ... ์ œ๋Œ€๋กœ ์ž‘๋™ํ–ˆ์–ด์•ผ ํ–ˆ์Šต๋‹ˆ๋‹ค ๐Ÿค” ์–ด์ฉŒ๋ฉด ์šฐ๋ฆฌ๋Š” ๊ทธ๊ฒƒ์„ ์–ด๋–ป๊ฒŒ ํ™œ์„ฑํ™”ํ•ด์•ผ ํ• ์ง€๋„ ๋ชจ๋ฆ…๋‹ˆ๋‹ค

https://github.com/rnystrom/GitHawk/blob/master/Podfile.lock#L17 -L22

์˜ค, ์ด์ƒํ•˜๋„ค์š”. ๋ชฐ๋ผ.

@BasThomas ์ž‘๋™ ํ•˜๋ ค๋ฉด ANIMATED_GIF_SUPPORT ๋ฅผ ์ปดํŒŒ์ผ๋Ÿฌ ํ”Œ๋ž˜๊ทธ๋กœ ์ถ”๊ฐ€ํ•ด์•ผ ํ•˜๋Š” ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค. 5๋ถ„๋งŒ ์žˆ์œผ๋ฉด ๐Ÿ˜‰

https://github.com/NYTimes/NYTPhotoViewer/blob/master/NYTPhotoViewer/NYTScalingImageView.m#L21

์ง€๊ธˆ ๋‹ซ์„ ์ˆ˜ ์žˆ์Šต๋‹ˆ๊นŒ, @rnystrom?

๋‚˜์—๊ฒŒ "๊ทธ๋Ÿฐ ํŒŒ์ผ์ด๋‚˜ ๋””๋ ‰ํ† ๋ฆฌ"๋ฅผ ์ œ๊ณตํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค @Sherlouk ๐Ÿ˜‚
๋‚ด๊ฐ€ ๋ฌด์—‡์„ ๋†“์น˜๊ณ  ์žˆ์Šต๋‹ˆ๊นŒ?

screen shot 2017-10-13 at 10 11 05
screen shot 2017-10-13 at 10 11 08

์ดํ•ดํ•  ์ˆ˜ ์—†๋Š” ์ด์œ ๋กœ -D ์ ‘๋‘์‚ฌ๋ฅผ ๋ถ™์—ฌ์•ผ ํ•˜๋ฏ€๋กœ -DANIMATED_GIF_SUPPORT

๐Ÿค”, https://github.com/NYTimes/NYTPhotoViewer/blob/14458e8ed8065c3cf31fe0744db9fd4c506e4cfe/NYTPhotoViewer.podspec#L25 ์—†์ด๋„ ์ด๋ฏธ ์ž‘๋™ํ•˜๋Š” ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค.

์šฐ๋ฆฌ๊ฐ€ ํ•˜์œ„ ์‚ฌ์–‘์„ ์ง€์ •ํ•˜์ง€ ์•Š๊ธฐ ๋•Œ๋ฌธ์— ์ดํ•ดํ•  ์ˆ˜ ์—†์Šต๋‹ˆ๋‹ค: https://github.com/rnystrom/GitHawk/blob/master/Podfile#L12

๊ทธ๋ฆฌ๊ณ  ๊ธฐ๋ณธ ํ•˜์œ„ ์‚ฌ์–‘์„ ์ง€์ •ํ•˜์ง€ ์•Š์Šต๋‹ˆ๋‹ค ๐Ÿค” NYTPhotoViewer/AnimatedGifSupport ์ง์ ‘ ์ฐธ์กฐํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๊นŒ?

๋˜ํ•œ @BasThomas https://bugs.swift.org/browse/SR-909 -- ์ ์šฉ ๊ฐ€๋Šฅํ•œ๊ฐ€์š”?

๊ทธ๋ ‡๊ฒŒ ๋ณด์ด์ง€ ์•Š์Šต๋‹ˆ๋‹ค. #if ANIMATED_GIF_SUPPORT ๋Š” ์‚ฌ์‹ค์ž…๋‹ˆ๋‹ค.

ObjC์— ์ต์ˆ™ํ•˜์ง€ ์•Š์€๋ฐ #ifdef ๊ฐ€ ๋‹ค๋ฅด๊ฒŒ ์ฒ˜๋ฆฌ๋˜๋‚˜์š”?

https://github.com/NYTimes/NYTPhotoViewer/blob/master/NYTPhotoViewer/NYTScalingImageView.m#L125 ๋กœ

์ด๊ฒƒ์„ #75๋กœ ์˜ฎ๊ธฐ์ž

์ด ํŽ˜์ด์ง€๊ฐ€ ๋„์›€์ด ๋˜์—ˆ๋‚˜์š”?
0 / 5 - 0 ๋“ฑ๊ธ‰