Material Design Stock Theme


(Erlend Sogge Heggen) #224

No worries, that’s why PRs are so great since maintainers get to review it before applying it. So by all means give it a try and we’ll guide you along as necessary.

You can either:

  1. Make a pull request from your own branch, or;
  2. Edit via the front-end, which creates a branch for you.

Option two is easier and pretty handy when you only need to edit one file.

(do note it seems GitHub might be having some server issues right now)


(James North) #225

Awesome, thanks @erlend_sh!

I think I’ve done the desktop fix with method 2.

Might add a sneaky request to add the mobile cards thing too :slight_smile:

I feel like I’ve finally contributed something - thanks again!


(Joshua Rosenfeld) #229

Hi @JamesNorth, thanks for your PRs! Wanted to share some thoughts here.


Thanks for tweaking your PR to avoid duplication of CSS rules!

As a whole the Material Design Theme does not make changes on mobile, with the exception of font. Seems odd to add support to the topic view (cards) without tweaking the remainder of the mobile UI. Do you have screenshots of what the mobile UI will look like?


(Sam Saffron) #230

@rewphus would you mind reviewing and (possibly closing what needs closing) the 4 PRs in Pull Requests · discourse/material-design-stock-theme · GitHub ?


(James North) #231

Thanks for your help and suggestions to clean it up @jomaxro!

I didn’t bother (but could) carding the topic list, for mobile view but the conversation cards look like this:

Certainly the location of the avatar can improve at least.


#232

Yeah, I will do what I can to address and clean up the PRs but might not be able to get to it until Wednesday. Thanks to everyone who has contributed.


(xiasummer) #233

Hello, I found something interesting, when I adopted this theme. For a new post when I want to click the left bottom button manage the post, seems to be not working?

image

I checked again. When I change into the default Theme, it will be functional, but when I change into this theme, it will be useless clicking such button. So I believe there must be sth wrong with the theme.

And, could you please tell me which code decides the animation of (blue) light when creating a new post? Both in the post panel(the main body) and in the list panel(The post name below the “Latest” tag)? I want to delete such animation.

Here is the Theme Code

Yours


(Daniela) #234

Do you mean the administration key button? It works for me with this theme. Do you see any errors in the browser console when you try to click it?

Try to see here:


(xiasummer) #235

administration key button works fine in my windows 7 computer, I think it might be sth wrong witn my windows 10 system.

about the animation, I will learn how to edit it.

Thank you.


#236

thanks for all of your hard work on this theme!


(Bhanu Sharma) #237

Screenshot-2017-11-14 BeatParadise Forum(1)

Hi! … is there a way to fix this padding issue? I can’t seem to be able to fix it.


(Daniela) #238

For online users?
#whos-online { padding-bottom: 5px; }


(Bhanu Sharma) #239

Hey Thanks!
That worked like a charm


#240

How can I fix this error?

Thank you for this amazing theme! :slight_smile:


(Sam Saffron) #241

You mean the Log In padding, @rewphus do you have time to look? If not @awesomerobot can do a quick fix.


#242

Should be fixed now. Update theme and let me know if you still see any issues.


(David Taylor) #243

I’ve just added this CSS to the plugin by default, so @itsbhanusharma once you update the whos-online plugin you can remove the CSS modification :slight_smile:


(Bhanu Sharma) #244

Sure! … Doing it now!


(Uncle) #245

Love the theme. Please can someone help me to add some border-radius

See the places where I am trying to curve the borders.

Many thanks folks.

Cheers


#246

For the alert info, this should do the trick :

.alert.alert-info {
    border-radius: 5px;
}

For the breadcrumb :

ol.category-breadcrumb {
    border-radius: 5px;
}

For the topic-list, it should be something like this but I didn’t try

.topic-list {
    border-radius: 10px;
}

You an play with the size of the border radius for a bigger curve