Feat : 웹 뷰 디자인 수정 및 개인정보 처리방침,문의하기 웹 뷰로 출력

[m6z1]

📮 관련 이슈

• resolved 웹 뷰 디자인 수정 #278

✍️ 구현 내용

• 공지사항 웹 뷰의 닫기 버튼을 우측으로 이동했습니다.
• 개인정보처리방침, 문의하기 버튼 클릭 시 외부 페이지가 아닌 웹 뷰를 이용합니다.

📷 구현 영상

✔️ 확인 사항

• 컨벤션에 맞는 PR 타이틀
• 관련 이슈 연결
• PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
• Github Action 통과

[huiwoo-jo]

수고하셨습니다!
리뷰 확인 부탁드려요🙇‍♂️

[huiwoo-jo]

연속 클릭 시 WebActivity가 여러 번 열릴 가능성이 있어 방지하는 부분이 있으면 좋을 것 같아요

[huiwoo-jo]

카카오톡으로 보내드렸던 자료 첨부합니당
https://ju-hy.tistory.com/46

[huiwoo-jo]

컨벤션 맞춰주세요!

Suggested change

	override fun initDataBinding() {
	}
	override fun initAfterBinding() {
	}
	override fun initDataBinding() = Unit
	override fun initAfterBinding() = Unit

[huiwoo-jo]

이렇게 수정하면 ConstaintLayout을 두개를 쓸 필요가 없을거 같은데 두개를 쓰신 이유가 따로 있나요?

Suggested change

	<?xml version="1.0" encoding="utf-8"?>
	<layout xmlns:app="http://schemas.android.com/apk/res-auto">
	<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
	android:layout_width="match_parent"
	android:layout_height="match_parent">
	<androidx.constraintlayout.widget.ConstraintLayout
	android:id="@+id/cl_web_top"
	android:layout_width="0dp"
	android:layout_height="wrap_content"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintStart_toStartOf="parent"
	app:layout_constraintTop_toTopOf="parent">
	<ImageView
	android:id="@+id/btn_web_close"
	android:layout_width="wrap_content"
	android:layout_height="wrap_content"
	android:layout_marginVertical="4dp"
	android:layout_marginEnd="16dp"
	android:clickable="true"
	android:contentDescription="@null"
	android:focusable="true"
	android:padding="8dp"
	android:src="@drawable/ic_close"
	app:layout_constraintBottom_toBottomOf="parent"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintTop_toTopOf="parent" />
	</androidx.constraintlayout.widget.ConstraintLayout>
	<WebView
	android:id="@+id/wv_web"
	android:layout_width="0dp"
	android:layout_height="0dp"
	app:layout_constraintBottom_toBottomOf="parent"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintStart_toStartOf="parent"
	app:layout_constraintTop_toBottomOf="@id/cl_web_top" />
	</androidx.constraintlayout.widget.ConstraintLayout>
	</layout>
	<?xml version="1.0" encoding="utf-8"?>
	<layout xmlns:app="http://schemas.android.com/apk/res-auto">
	<androidx.constraintlayout.widget.ConstraintLayout
	xmlns:android="http://schemas.android.com/apk/res/android"
	android:layout_width="match_parent"
	android:layout_height="match_parent">
	<ImageView
	android:id="@+id/btn_web_close"
	android:layout_width="wrap_content"
	android:layout_height="wrap_content"
	android:layout_marginVertical="4dp"
	android:layout_marginEnd="16dp"
	android:clickable="true"
	android:contentDescription="@null"
	android:focusable="true"
	android:padding="8dp"
	android:src="@drawable/ic_close"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintTop_toTopOf="parent" />
	<WebView
	android:id="@+id/wv_web"
	android:layout_width="0dp"
	android:layout_height="0dp"
	app:layout_constraintBottom_toBottomOf="parent"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintStart_toStartOf="parent"
	app:layout_constraintTop_toBottomOf="@id/btn_web_close" />
	</androidx.constraintlayout.widget.ConstraintLayout>
	</layout>

[m6z1]

ㅇ�오 아뇨 ㅎ 복붙해와서 그런 것 같아요 감사합니다 👍

[huiwoo-jo]

머지합시다🚀