Skip to content

PNG파일의 섬네일 생성시 배경이 투명할 수 있도록 수정#2228

Closed
lalafellsleep wants to merge 4 commits into
xpressengine:developfrom
lalafellsleep:develop-alpha-thumb
Closed

PNG파일의 섬네일 생성시 배경이 투명할 수 있도록 수정#2228
lalafellsleep wants to merge 4 commits into
xpressengine:developfrom
lalafellsleep:develop-alpha-thumb

Conversation

@lalafellsleep

Copy link
Copy Markdown
Contributor

image

  • 서드파티 및 Member모듈의 insertProfileImage에서 사용할 수 있도록 확장

  • 기본값: N

@lalafellsleep

Copy link
Copy Markdown
Contributor Author

파일 업로드에 들어간 설정은 들어갈 위치가 마땅치 않아 임시적으로 넣어두었습니다.

해당 설정을 무시하고 서드파티에서도 컨트롤 할 수 있지만 이러한 설정의 전역 설정(?)이 없는것이 어색할 수 있다고 생각하여 추가하였으나 file모듈의 설정부분이 불필요하다고 판단되는 경우 제외하셔도 무방합니다.

@ghost ghost self-assigned this Mar 13, 2018
@ghost ghost added the type/enhancement label Mar 13, 2018
@ghost

ghost commented Mar 19, 2018

Copy link
Copy Markdown

말씀하신 것처럼 allow_thumbnail_transparent 설정을 사용하는 곳이 없는 상태이고,
FileHandler::createImageFile()를 호출할 때 결정되므로 이를 사용하는 곳에서 결정될 부분인 것 같습니다.
설정 부분을 제거해주시거나, 여의치 않으시면 제가 제거하여 반영하도록 하겠습니다.

@ghost ghost added this to the 1.10.0 milestone Mar 19, 2018
@lalafellsleep

lalafellsleep commented Mar 19, 2018

Copy link
Copy Markdown
Contributor Author

@bnu 확인했습니다. 우선 해당 사항을 분리하고

회원 모듈의 프로필 업로드 시 배경 투명 허용 옵션으로 적용하여 별도 pr 하도록 하겠습니다.

@ghost ghost removed this from the 1.10.0 milestone Mar 20, 2018
@ghost

ghost commented Mar 20, 2018

Copy link
Copy Markdown

#2243 PR에서 진행

@ghost ghost closed this Mar 20, 2018
@lalafellsleep lalafellsleep deleted the develop-alpha-thumb branch March 20, 2018 14:27
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant