[Nextcloud/android] passcode 레이아웃 정렬 수정 PR -> Merge

 💚이전글 읽기💚




passcode 레이아웃 정렬을 수정한 후 날린 PR#7370에 멤버 개발자 AndyScherzinger가 to review 레이블을 달아줬다.😍 그리고 다른 멤버 개발자 tobiasKaminsky에게 리뷰를 요청했다.

tobiasKaminsky는 중앙 정렬이 아닌 왼쪽 정렬을 한 것이 더 일반적이라는 코멘트와 함께 AndyScherzinger와 nextcloud/designers를 태그 걸어 어떻게 생각하는지 물어봤다. 나는 중앙 정렬이 더 예쁘다고 생각했는 데,, 디자인은 너무 어렵다.😂😂

AndyScherzinger는 타이틀은 왼쪽정렬, PIN은 중앙정렬, 취소버튼은 오른쪽 정렬이어야 한다고 코멘트를 남겼다. 그리고 현재와 같은 디자인이 아닌 어두운 배경에 둥근 네모 상자에 이들을 표시하는 게 좋다고 했다. 현재는 하얀 배경에 요소들이 표시되어있는데, AndyScherzinger의 추가 의견은 괜찮다고 생각한다.

Jancborchardt도 코멘트를 남겨줬다. 사실 Nextcloud/android에서 처음 보는 개발자라서 우리처럼 이슈 관리하고 있는 개발자인가 했는데 멤버 개발자표시가 있었다!! 프로필을 보니 오픈소스 디자이너였다! 아까 태그걸린 nextcloud/designers와 관련있는 사람인가보다. 디자이너...멋있다..👍🏻

내 PR을 nice enhancement라고 해줬다. 감동...😭😭😭

Jancborchardt는 코멘트를 정말 깔끔하게 정리했다. 우선 전제화면이아닌 다이얼로그처럼 보이게 하자는 AndyScherzinger 의견에 동의했다. 취소 버튼은 이곳에서 주가 아니게 때문에 기본으로 스타일을 지정해서는 안되고 중앙에 정렬해야 한다고 한다. 

Jancborchardt는 디자인을 추가로 수정하는 것은 새로운 이슈로 옮기자고 했고, 병합되었다! 이전 의견까지는 PR이 안 받아지겠구나 생각했는데 다행이다..ㅎㅎ 새로운 이슈에도 참여하고 싶었지만 사실 멤버 개발자들이 원하는 다이얼로그 형식의 비밀번호를 어떻게 만들어야할지 모르겠어서 참여하지 않았다. 디자인은 너무너무 어렵다.😵

#7403에서는 중앙 정렬로 변경된 디자인을 테스트할 수 있는 apk 파일이 있다.


AngyScherzinger는 마지막에 이렇게 코멘트를 남겨준다.!! :-)




una.kwon Github: https://github.com/Kuuuna98
Nextcloud/android Github: https://github.com/nextcloud/android
 

댓글

  1. 이인재2020년 11월 21일 오전 8:36

    와!!!!!! 병합 축하드려요!!
    중앙정렬이 더 깔끔해보이는데 디자이너님이 동의해주셔서 다행입니다 ^^
    고생하셨습니다!!

    답글삭제

댓글 쓰기

이 블로그의 인기 게시물

[Nextcloud/android] 배터리 최적화 제외 여부 판단 오류 (Issue #7304)

이미지
  이 것 또한 Nextcloud를 직접 사용해보면서 발견한 오류다. 사실 어플리케이션을 처음 사용하면서 겪었던 오류는 #7273이었다. 자동 업로드 설정을 키니까 매번 존재하는 파일을 덮어씌우는 것이다...😩 업로드 알림이 계속 뜨니까 너무 보기 싫었다 관련된 이슈가 있을까 싶어 찾아보던 중 이 오류를 발견했다! 멤버 개발자인 tobias Kaminsky는 자동 업로드 설정 기능으로 해결할 수 있다고 답변했고, 이슈를 닫아버렸다 이 설정 기능으로 자동업로드 옵션을 변경할 수 있었다! 자동업로드 옵션 하지만... 설정은 변경되지 않았고 배터리 최적화 대상에서 Nextcloud를 제외했음에도 배터리 최적화 설정을 변경해야 한다는 알림창이 뜨는 것이다😭😭 혹시 저 비활성화 버튼이 자동업로드 설정 변경을 비활성화 하는 건가 싶어 코드까지 찾아보았다.           src>main>res>values-ko>strings.xml '비활성화'는 배터리 최적화 대상에서 제외한다는 의미다.          src>main>java>com>owncloud>android>ui>activity>SyncedFoldersActivity.java  위의 코드를 살펴보면 '비활성화' 버튼 클릭 시, SDK 에 따라 배터리 최적화를 풀거나 배터리 파워 사용량을 보이는 액션이 있어야 한다. 하지만 내 환경에선 어떠한 액션도 보이지 않고 다이얼로그 창을 닫아버린다.. 이에 대해 새로운 이슈를 제출하기로 했다! 재현은 디바이스 설정에서 Nextcloud를 배터리 최적화 대상에서 제외한 이후에, 자동업로드 설정을 변경하여 이슈 재현을 확인할 수 있다. 멤버 개발자분들이 영어로 소통하시는 것을 감안해, 디바이스의 언어 설정을 English로 바꾼 후 스크린샷을 추가로 첨부했다😀 첨부한 스크린샷 이에 대해 tob...
자세한 내용 보기

[Nextcloud/android] passcode 화면에서 뒤로가기 이벤트 PR -> Merge

이미지
   💚이전글 읽기💚 이슈 등록 (Issue #7343):  https://cnu-devapp.blogspot.com/2020/11/nextcloudandroid-pin-issue-7343.html PR 날리기:  https://cnu-devapp.blogspot.com/2020/11/pin-pr-issue-7372.html #7372  PR을 날린 후 멤버 개발자 AndyScherzinger가 to review 레이블을 추가해줬다. 1 2 3 4 5 6 7 8 9 10 11 12 13 14 @Override public   boolean  onKeyDown( int  keyCode, KeyEvent event){      if  (keyCode  = =  KeyEvent.KEYCODE_BACK  & &  event.getRepeatCount() = =   0 ){          if (ACTION_CHECK. equals (getIntent().getAction())){             moveTaskToBack( true );             finishAndRemoveTask();         } else   if  (ACTION_REQUEST_WITH_RESULT. equals (getIntent().getAction())  | |   ...
자세한 내용 보기

[Nextcloud/server, android] 개발 환경 구성

이미지
  기여에 앞서서, 먼저 환경 구성을 해야 한다. Nextcloud/android 레포지토리에 기여하기로 했지만, 개발 환경에 서버도 필요하기 때문에 Nextcloud/server도 구축해야 한다. 1. Nextcloud Server 구축 서버 설치 및 구성은 아래 문서를 참고하였다. https://docs.nextcloud.com//server/stable/admin_manual/installation/index.html 나는 Nextcloud server 운영체제로 Ubuntu 20.04 LTS를 선택했으며, 데이터베이스는 MySQL을 사용하여 구축할 것이다. 먼저 가상 머신에 Ubuntu 20.04 LTS를 설치한 후, 아래 사이트에서 nextcloud-20.0.1을 다운받는다. https://nextcloud.com/install/ 서버 구축에 필요한 라이브러리 설치 아래 커맨드를 통해 필요한 apache, mysql(mariadb), php를 설치한다 sudo apt update sudo apt install apache2 mariadb - server libapache2 - mod - php7 . 4 sudo apt install php7 . 4 - gd php7 . 4 - mysql php7 . 4 - curl php7 . 4 - mbstring php7 . 4 - intl sudo apt install php7 . 4 - gmp php7 . 4 - bcmath php - imagick php7 . 4 - xml php7 . 4 - zip 위의 mariadb-server를 설치하다가 아래와 같은 오류가 발생했다. MariaDB에 종속성이 없거나 패키지가 손상되었다는 오류다..😅 The following packages have unmet dependencies: mariadb-server : Depends: mariadb-server-10.3 but it is not going to be ins...
자세한 내용 보기