Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[AN] 스레드 뷰 입력 편의성 향상 및 TextView 링크 클릭 및 입력 커서 테마 적용 #598

Merged
merged 8 commits into from
Nov 4, 2023
Merged

[AN] 스레드 뷰 입력 편의성 향상 및 TextView 링크 클릭 및 입력 커서 테마 적용 #598

merged 8 commits into from
Nov 4, 2023

Conversation

no1msh
Copy link
Collaborator

@no1msh no1msh commented Nov 1, 2023

😋 작업한 내용

그동안의 불편한 점이 많았던 스레드 뷰의 불편사항을 해결하였습니다.

  • 스레드 입력 시 키보드 줄바꿈을 추가합니다.
  • 스레드 입력 시 전송 버튼을 누르는 순간 키보드가 내려가는 현상을 수정합니다.
  • 스레드 채팅창에서 입력할 때 보이는 미리보기를 4줄까지 확인할 수 있게 합니다.
  • 여러줄 입력 시 스크롤로 처음과 끝을 확인할 수 있게 합니다.
  • 스레드 인증 뷰에서 EditText를 클릭하면 키보드가 올라오고 붙여넣기를 하려고 EditText를 다시 클릭하는 순간 키보드가 내려가는 현상을 수정합니다.
  • 입력 커서 테마가 기본 머티리얼 색상으로 지정되었던 현상을 수정하였습니다.

🙏 PR Point

사진이 첨부되질 않아 슬랙으로 스크린샷 첨부하였습니다.

👍 관련 이슈

@no1msh no1msh added android💚 안드 feat 기능 추가 fix 기능 코드 버그 수정 ßändäl🌗 반달 labels Nov 1, 2023
@no1msh no1msh self-assigned this Nov 1, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 1, 2023 •
edited
Loading

Test Results

9 tests   9 ✔️  0s ⏱️
3 suites  0 💤
3 files    0 ❌

Results for commit 7489804.

♻️ This comment has been updated with latest results.

RightHennessy
RightHennessy approved these changes Nov 2, 2023
View reviewed changes
Copy link
Collaborator

@RightHennessy RightHennessy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

뀻뀻 고생하셨습니다.

inseonyun
inseonyun approved these changes Nov 2, 2023
View reviewed changes
Copy link
Member

@inseonyun inseonyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lon
G
Ti
Me no see

android/app/src/main/res/values/themes.xml
Comment on lines +20 to +22
<item name="android:textColorLink">@color/green02_E639D353</item>
<item name="android:textColorHighlight">@color/green04_26A641</item>
<item name="colorControlActivated">@color/green04_26A641</item>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이렇게 세부적으로 줄 수 있군요 배워갑니당

@no1msh no1msh merged commit fe3f5f7 into develop Nov 4, 2023
2 checks passed
@no1msh no1msh deleted the AN/feature/589-thread branch November 4, 2023 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RightHennessy RightHennessy RightHennessy approved these changes

@inseonyun inseonyun inseonyun approved these changes

Assignees

@no1msh no1msh

Labels
android💚 안드 feat 기능 추가 fix 기능 코드 버그 수정 ßändäl🌗 반달
Projects
2023-yigongil
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[AN] 스레드(피드)에서 사용자 편의성을 증가시킵니다.
3 participants
@no1msh @inseonyun @RightHennessy