Minor redesign #612

Merged

btzr-io merged 18 commits from css_patch into v17-redesign

2017-09-30 03:07:22 +02:00

btzr-io commented

2017-09-25 04:35:47 +02:00

(Migrated from github.com)

A bunch of minor CSS fixes and additional changes.

Changes

Update themes.
Tabs (redesign) #576
Custom styled scrollbar #574
Add more focus on search.
Update changelog.

Fixes

Fix: unresponsive header #613
Fix: Dark theme issues
Minor Fix: https://github.com/lbryio/lbry-app/issues/620
Minor css fixes.

Todo

~~Update changelog.~~

Removed

This fixes were removed because they require a lot of changes and thinking
due to the current poorUI / UX 😛

This would be handle in their own PR or the main one -> #598

~~Init implementation of empty states #551~~
~~Fix: Page titles in search bar don't stand out enough #252~~
~~Thumbnail Resizing In Search/Browsing #595~~

Actually this isn't a css bug, see #620, #551

~~UI glitch after accessing a video without enough credits #605~~

A bunch of minor `CSS` fixes and additional changes. ### Changes - [x] Update themes. - [x] Tabs (redesign) #576 - [x] Custom styled scrollbar #574 - [x] Add more focus on search. - [x] Update changelog. ### Fixes - [x] Fix: unresponsive header #613 - [x] Fix: Dark theme issues - [x] Minor Fix: https://github.com/lbryio/lbry-app/issues/620 - [x] Minor css fixes. ### Todo ~Update changelog.~ ### Removed This fixes were removed because they require a lot of changes and thinking due to the current poor`UI / UX` :stuck_out_tongue: This would be handle in their own PR or the main one -> #598 > ~Init implementation of empty states #551~ > ~Fix: Page titles in search bar don't stand out enough #252~ > ~Thumbnail Resizing In Search/Browsing #595~ Actually this isn't a css bug, see #620, #551 > ~UI glitch after accessing a video without enough credits #605~

kauffj (Migrated from github.com) reviewed

2017-09-25 04:35:47 +02:00

MSFTserver commented

2017-09-27 04:10:07 +02:00

(Migrated from github.com)

should add in something where if the field for licence is blank to not display it or display none instead of a blank licence area

👍 1

btzr-io commented

2017-09-29 04:33:50 +02:00

(Migrated from github.com)

fixed conflicts https://github.com/lbryio/lbry-app/pull/612/commits/2e285822774a995d6cbbc2b9f3713b72947f6c4d

btzr-io commented

2017-09-30 00:01:57 +02:00

(Migrated from github.com)

@kauffj if you aren't ok merging this in master I can move it to redesign,
but this was intended to be a css-patch for the next release. 😛

@kauffj if you aren't ok merging this in master I can move it to redesign, but this was intended to be a `css-patch` for the next release. :stuck_out_tongue:

kauffj commented

2017-09-30 02:34:26 +02:00

(Migrated from github.com)

I expect to put this into 0.17 release

On Sep 29, 2017 6:01 PM, "Baltazar Gomez" notifications@github.com wrote:

@kauffj https://github.com/kauffj if you aren't ok merging this in
master I can move it to redesign, but this was intended to be a css-patch
for the next release. 😛

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry-app/pull/612#issuecomment-333249356, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgZVgvKGDJBLRpMZysbJJEqFa_Bvavlks5snWjWgaJpZM4PiIvj
.

I expect to put this into 0.17 release On Sep 29, 2017 6:01 PM, "Baltazar Gomez" <notifications@github.com> wrote: > @kauffj <https://github.com/kauffj> if you aren't ok merging this in > master I can move it to redesign, but this was intended to be a css-patch > for the next release. 😛 > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/lbryio/lbry-app/pull/612#issuecomment-333249356>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AAgZVgvKGDJBLRpMZysbJJEqFa_Bvavlks5snWjWgaJpZM4PiIvj> > . >

👍 1

btzr-io commented

2017-09-30 03:06:25 +02:00

(Migrated from github.com)

Ok then I guess we have more time to add more stuff ^^