새로운 재생 목록 플레이어와 평론 스타일을 실현하다

7939 단어 alonetone

묘사

수정 #939, #938
수달라와 토론:
*[] 새 버튼의 스크롤 상태
*[x] 저자: 아티스트 논리
*[x] 즐겨찾기에 추가된 애니메이션이 올바르게 삭제됨
*[x] [트랙 편집]의 [삭제] 버튼 처리 방법
*[x] 재생 목록 트랙 아래에 있는 설명 그룹을 처리하는 방법
*[x] 체크 상자의 크기가 다른데 사이트 범위 내에 변화가 있습니까?
*[x] 재생 목록 편집 및 설명 편집 버튼에 새 위치 추가하기 고려
*[] 외부 링크의 색상 테마

"검토 준비" 목록


이 검사표들은 코드 심사에 대한 기초 지식을 확보하는 데 도움이 된다.소음 제거를 고려하다.
  • [x]PR 제목의 정확한 요약 변경
  • [독립된 방법이나 새로운 종점에 대한 새로운 시험 증가
  • [] 논리에 맞는 후속 행동에 대해 질문
  • 을 열었습니다.
  • [] 만약 이것이 버그를 복구했다면 "fixes#XXX"는 설명의 첫 줄이나 마지막 줄입니다.
  • 코드 심사 전과 심사 기간의 기타 제출 후.

  • [제목 및 설명을 업데이트하여 추가 변경 사항을 설명합니다
  • [모든 테스트는 녹색
  • [로컬 마이그레이션 테스트와 올바른 작업
  • [] 로컬에서 지사를 시작하고 새 코드를 실행합니다
  • .
  • [] 포시의 변화는 목적이 있거나 해석을 거친 것이다
  • [] 핸드폰의 Css 변경이 만족스럽다(Percy를 통과하면 된다)
  • 토론 #1

    85524 행의 PR diff 크기는 인라인 주석 기능이 허용하는 최대치를 초과합니다.

    토론 #2

    Rollover states on any of these new buttons


    image
    새 즐겨찾기/다운로드 버튼 말입니까?

    by You by Artist logic


    우리는 현재 상태를 유지해야 합니까?

    what to do with the delete button on track edit


    곡 편집을 변경해야 하나요?어둠의 주제.

    add to favorites animation properly removed


    응, 이게 무슨 뜻인지 모르겠어.우리 애니메이션 잃어버렸어?우리는 당연히 그것을 버튼 위에 놓고 싶지만, 사이즈에 문제가 있으면 그것을 뒤로 밀어야 한다.

    how to deal with groups of comments below a playlist track


    지금의 스타일을 유지하는 게 어때요? 화살표를 사용하면 사모의 목록에 놓을 수 있어요.

    checkboxes are different sizes, site wide change?


    나의 투표 결과, 우리는 모든 곳에서 새로운 16x16을 사용할 것이다.보기에 주로 재생 목록 편집과...

    consider new placements for edit playlist and edit descriptions buttons


    재생 목록 편집이 제목 바로 아래의 편집으로 변경되었습니다.단추를 보류해야 합니까? 아니면 제거할 수 있습니까?나는 잠시 묘사라는 단어를 숨길 수 있다.

    color themes for external links


    트랙 텍스트에서?그들은 정적 페이지의 풍격을 따라야 한다.

    토론 #셋

    의 한 가지 선택은 지점이 안정적인 ish 상태에 있다고 생각하면 우리는 주 제어를 통합하고 충돌을 해결한 다음에 이 항목들을 합병하여 새로운 문제에 추가할 수 있다는 것이다.

    토론 #4

    Do you mean the new favorite / download button?


    네, 특히 그렇습니다. 그러나 재생 목록 디자인의 다른 단추는 구성 요소를 사용하지 않았습니다. 편집과 댓글을 발표할 때 사용하는orangetone 배경은 표준적인orange600 기본 단추가 아닙니다.

    by You by Artist logic


    우리는 현재 상태를 유지해야 합니까?
    나는 지금 그것이'네가 해'라고 말하지 않아서 매우 기쁘다!

    what to do with the delete button on track edit Are we changing track edit? Or you mean re: dark theme.


    이 부분이 변경되었기 때문에 삭제 단추의 레이아웃을 잠시 중단합니다.단지 그 위치를 다시 고려할 기회일 뿐이다.하지만 안정되어 앉아서 이렇게:
    Screenshot 2019-12-02 17 18 17

    add to favorites animation properly removed Hmm, not sure what this means. Did we lose the animation? We definitely want to keep it in the button and should push back if there's a size problem.


    애니메이션이 축소되어 단추에 남아 있을지 모르겠지만, 지금은 다시 단추로 돌아왔습니다.

    how to deal with groups of comments below a playlist track


    지금의 스타일을 유지하는 게 어때요? 화살표를 사용하면 사모의 목록에 놓을 수 있어요.
    나는 이 위의 삼각형을 단선 보기의 주석 그룹에 다시 추가했다.

    checkboxes are different sizes, site wide change?


    나의 투표 결과, 우리는 모든 곳에서 새로운 16x16을 사용할 것이다.보기에 주로 재생 목록 편집과...
    나는 형식적으로 변화를 했다.따라서 모든 곳에 적용될 것이다.

    consider new placements for edit playlist and edit descriptions buttons


    재생 목록 편집이 제목 바로 아래의 편집으로 변경되었습니다.단추를 보류해야 합니까? 아니면 제거할 수 있습니까?나는 잠시 묘사라는 단어를 숨길 수 있다.
    나는 편집 설명을 그다지 좋아하지 않아서, 나는 잠시 그것을 삭제했다.그 작은 버튼 부분은 이미 존재하지 않는다.제목과 예술가의 이름에 따라 재생 목록을 편집합니다.

    color themes for external links


    트랙 텍스트에서?그들은 정적 페이지의 풍격을 따라야 한다.
    이것은 사이드바 밑부분이나 표지 보기의 외부 링크와 관련이 있습니다.최신 Figma 흑백 재생 목록에는 존재하지 않지만 읽을 수 있는 색을 대충 선택했습니다.

    토론 #5

    가 마지막 몇 가지 일일 수도 있다.
  • [x] 단일 트랙'단추 표시줄'위에 공백이 있어야 합니다. 한 트랙에 설명이 있든 없든.호흡이 필요한 공간...
  • 0p1coast live at melatonin by carole | alonetone 2019-12-03 21-41-52
    image

  • [x] 우리는 단일 보기에 예술가의 이름을 설정하지 않았다.현재 위치와 텍스트 스타일이 부작용을 일으켜 '메타데이터 표시줄' 을 깨끗하게 보이지 않게 합니다.나는 예술가의'속'제목을 확정하고 h1에서 더 가까운 두 픽셀로 옮기고 h1의 간격samo를 사용하여'제목과 작가'그룹의 공간에 사용해야 한다고 생각한다.기왕 그것이 이미 더 작은 텍스트가 된 이상, 아마도 우리도 작가의 텍스트를 검은색으로 바꿀 수 있을 것이다.기본적으로 제목 그룹과 메타데이터 슬롯 그룹을 시각적으로 분리하는 방법을 찾기 위해 현재 작가는 그것들을 분리했는데 결과는 매우 바빠 보인다.또 다른 선택은'Sudara Upload on X'와 같은 작성자를 메타데이터로 사용하는 것이지만, 이는 작성자를 지나치게 깎아내릴 수 있다.

  • [x] 다운로드/제일 좋아하는 아이콘이 좀 높은 것 같은데?그들은 나에 대한 감각이 좀 간지럽다.
  • 나는 싱글 페이지에 연예인을 추가할 것이다...

    토론 #6

  • [x] 나는 지금이 편집 궤도에서'삭제'붕괴를 처리하는 좋은 시기라고 생각한다.나는 '저장' 단추를 예상 위치로 이동하고 uhh로 업그레이드할 것이다.주 버튼 스타일(주황색, 더 큰)삭제는 '저장' 위치로 이동할 것입니다. 새 편집 프로필의 삭제 단추는 빨간색이어야 합니다.
  • alonetone 2019-12-03 22-07-04
    alonetone white theme 2019-12-03 22-09-14

    토론 #7

    불행하게도, 나는 우리가 재생 목록의 '편집' 단추를 아직 완성하지 못했다고 생각한다.우리의 모든 다른 편집 단추는 보기에 모두 다르다. 나는 우리가 어느 뚜렷한 곳에 같은 스타일의 단추를 추가해야 한다고 생각한다.하지만 나는 변화가 없는 상황에서 물건을 보내서 매우 기쁘다. 우리는 일의 진전을 볼 수 있다.

    토론 #8

    • [ ] I think now is a good time to handle that "delete" debacle on edit track. I'd move the "save" button to it's expected location and "upgrade" it to the uhh.... "primary" button style (orange, bigger). And I'd move delete where "save" is now and it should be red ala the delete button on the new edit profile.

    alonetone 2019-12-03 22-07-04

    alonetone white theme 2019-12-03 22-09-14


    @sudara에서 삭제 단추를 이동했지만, 이것은 나쁜 mass edit와 일부 travis 테스트입니다. 왜냐하면 이것은 현재 새 파일에서 assets/edit가 assets/edit가 아니라 assets/edit이기 때문입니다.

    토론 #9

    Unfortunately I think we are not done yet with the "edit" button on playlist. All of our other edit buttons look different, I think we may need to add one in the same style, somewhere obvious. But I'm happy shipping this without that change for now, we can see how things go.


    아마도 그것은 작은 텍스트 링크로서 좀 약할 것이다.전설은 계속된다!

    토론 #10

    I've moved the delete button, but it's broken mass_edit and some travis tests, because its in a new file now, assets/_edit instead of assets/edit?


    나는 가장자리를 제외하고 대량으로 편집된 백엔드 내용을 복구했다. mp3를 잃어버리면 오류가 발생할 수 있다.
    그러나 css의 경우 대규모 편집에 대한aax 성공 피드백을 잃은 것 같습니다.모든 자극 조치가 여전히 작용을 발휘하고 있다.

    토론 #11

    @sudaraAjax 피드백은 대량 편집에 다시 나타날 것입니다!

    좋은 웹페이지 즐겨찾기