joomla / joomla-cms

Home of the Joomla! Content Management System
https://www.joomla.org
GNU General Public License v2.0
4.79k stars 3.65k forks source link

[4.0] Choices css on filters #28272

Closed brianteeman closed 4 years ago

brianteeman commented 4 years ago

Sorry my scss-fu is not good enough to fix this - I can do some but not all. @angieradtke can you?

[ ] text color [ ] border-radius [ ] border color [ ] border color : focus [ ] box-shadow [ ] box-shadow : focus

before

angieradtke commented 4 years ago

select tags should like select access?

angieradtke commented 4 years ago

.choices.is-focused {border-color:#39f; box-shadow: 0 0 0 .2rem #d3e2f5;}

But.... it looks a little bit strange at the bottom

brianteeman commented 4 years ago

I have done most bits except for the part that makes the box-shadow when focused have rounded corners and not square ones - can't find that

jwaisner commented 4 years ago

Confirmed.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28272.

angieradtke commented 4 years ago

ah now I understand. administrator/templates/atum/scss/vendor/choicesjs/choices.scss

` .choices__inner { border: $input-border; border-radius: $border-radius; box-shadow: $input-box-shadow; }

.choices.is-focused { .choices__inner { border-color: $focuscolor; box-shadow: $focusshadow

} }

`

brianteeman commented 4 years ago

thanks - will check it later and submit pr

brianteeman commented 4 years ago

did you check that - It doesnt impact on the focus box-shadow corners - they are still square

angieradtke commented 4 years ago

border-radius: $border-radius; ??? I have that brian

brianteeman commented 4 years ago

@angieradtke I dont know how you get this to work

It cannot work because $border-radius is not defined in

// Atum Variables
@import "../../variables";
angieradtke commented 4 years ago

it's a bootstrap var media/vendor/bootstrap/scss/_variables.scss

brianteeman commented 4 years ago

well your code still doesn't work for me - odd

angieradtke commented 4 years ago

hm?

What does the inspector  says? Do you get an error message?

Am 10.03.2020 um 10:30 schrieb Brian Teeman:

well your code still doesn't work for me - odd

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/joomla/joomla-cms/issues/28272?email_source=notifications&email_token=AAI6HM4FNAGX7EIDNX2X5LDRGYCD7A5CNFSM4LDZNA7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOKU56I#issuecomment-596987641, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI6HMYDHENSFNFYZQ2FCBTRGYCD7ANCNFSM4LDZNA7A.

brianteeman commented 4 years ago

can you post an image of your successful inspector please

angieradtke commented 4 years ago

sure .-) brian

brianteeman commented 4 years ago

I still get the square corners

infograf768 commented 4 years ago

Folks. you are not talking about the same thing @angieradtke your patch does work by adding a rounded shadow when the choice is limited to one item (select the category when editing an article.) It does not work when the choices is multiple (searchtools)

angieradtke commented 4 years ago

I'm confused shot1 shot2

angieradtke commented 4 years ago

will update my branch

infograf768 commented 4 years ago

Select condition, select stage, select language etc, do not use fancyselect and choices.

Select category, Select author, Select Tag do (in searchtools), but they are multiple choices fields. Select category when editing an article is a single choice.

shadow

brianteeman commented 4 years ago

Based on the class names that @angieradtke mentioned she is talking about the correct place

infograf768 commented 4 years ago

The problem is not the class name. What I am trying to explain is that her patch works when fancy select is set to select-one, NOT when fancyselect is set to multiple... TBH I don't care about such petty issues. There are more important stuff to deal with.

brianteeman commented 4 years ago

Sorry you are completely wrong. And if you don't care then don't comment. You don't have to comment on everything

infograf768 commented 4 years ago

GL

angieradtke commented 4 years ago

it's a browser issue. Firefox has rounded corners, will take a look

angieradtke commented 4 years ago

the border-radius has to go to :

.choices { border: 0; border-radius:$border-radius; }