Please post screenshots Before and After pr.

Before

After

@infograf768

Can you please also style the summary element (to look like a button or an accordion header)?

Updated as you asked @dgrammatiko

#BEFORE

#AFTER

Can you use the same border-radius as in other places on template?

You have to check your PR, there are more changed files as needed. And the tests are failing (you can click on "Details" to see what is causing the problem)

The css files should not be commited

@hitesh-coder sorry your changes need to be applied only on this file https://github.com/joomla/joomla-cms/blob/4.0-dev/build/media_source/system/css/fields/joomla-image-select.css

@dgrammatiko but you asked to change summary element as accordion header and that should be done in respective file
so should i now revert all my changes and apply as before

@hitesh-coder the problem is that your change is global and not just on this one element

AFTER

I have tested this item ✅ successfully on b5f790a

Better with this styling. Should it have a border?

I have not tested this item.

I tried to Test this, but when I Apply Patch, I can't see any change.

@meinhoonharsh you can't test this using the patch tester, because the PR changes a scss file and for this you need to run npm run build:css to work

hey @ceford thanks for testing but could you please elaborate on the thing you mention I would love to make it better

@hitesh-coder

add a border

joomla-field-mediamore details {
    position: absolute;
    background: #f5f5f5;
    border-radius: .25rem;
    border: 1px solid #c9c9c9; // add this
}

to get

@meinhoonharsh you can't test this using the patch tester, because the PR changes a scss file and for this you need to run npm run build:css to work

Thank You @drmenzelit , I will make sure to do this from now

I have tested this item ✅ successfully on b5f790a

Before:

After:

hey @meinhoonharsh I have also added a border in my latest commit

I have tested this item ✅ successfully on 934c082

Yes - really nice with a border!

I have tested this item ✅ successfully on 934c082

I have Successfully tested this. It looks good in order to comparision with previous one

RTC

@hitesh-coder why did you close this PR?

@drmenzelit I mistakenly deleted the branch while exploring GitHub feature. can I make another PR?

@hitesh-coder No need to make a new PR. I've re-opened this so it can be merged. This can be done even when you branch was deleted, as far as I know.

Thanks!