? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
23 Feb 2020

Pull Request for Issue #28042.

Summary of Changes

Since Atum practically switched to rounded styling, we should use existing Bootstrap's styles instead of manually adding border radius everywhere.

Testing Instructions

Inspect forms with button groups and input groups.

Expected result

OK styling.

Actual result

Border radius added where not needed:

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 23 Feb 2020
avatar SharkyKZ SharkyKZ - change - 23 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2020
Category Administration Templates (admin) Layout
avatar brianteeman
brianteeman - comment - 23 Feb 2020

is it expected that now none of the corners are rounded

image

avatar SharkyKZ
SharkyKZ - comment - 23 Feb 2020

For now yes. Last button should actually have border radius on the right. But Create button is not last, there are some hidden buttons which are displayed with JS.

avatar SharkyKZ SharkyKZ - change - 24 Feb 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 24 Feb 2020
Category Administration Templates (admin) Layout Modules Administration Templates (admin) Layout
avatar infograf768
infograf768 - comment - 26 Feb 2020

Any specific page we can check with before/after ?

avatar SharkyKZ
SharkyKZ - comment - 26 Feb 2020

Mostly pages containing forms.

avatar infograf768
infograf768 - comment - 26 Feb 2020

I have tested this item successfully on aa57712


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

avatar infograf768 infograf768 - test_item - 26 Feb 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 26 Feb 2020

I have tested this item successfully on aa57712


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

avatar jwaisner jwaisner - test_item - 26 Feb 2020 - Tested successfully
avatar jwaisner jwaisner - change - 26 Feb 2020
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 26 Feb 2020

RTC


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

avatar jwaisner
jwaisner - comment - 26 Feb 2020

RTC


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

avatar rdeutz rdeutz - change - 29 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-29 08:23:55
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 29 Feb 2020
avatar rdeutz rdeutz - merge - 29 Feb 2020
avatar infograf768
infograf768 - comment - 29 Feb 2020

Found an issue in RTL for the search button
Screen Shot 2020-02-29 at 11 26 22

avatar infograf768
infograf768 - comment - 29 Feb 2020

@SharkyKZ
Can you solve the rtl display?

avatar SharkyKZ
SharkyKZ - comment - 29 Feb 2020

Looking into it.

avatar infograf768
infograf768 - comment - 29 Feb 2020

@SharkyKZ
I think I found the solution. I will have to create a specific rtl override in anew bootstrap file.
So don’t bother.

avatar infograf768
infograf768 - comment - 29 Feb 2020

Please test #28161
(Was easier to add css overrides in rtl template rather than creating a new scss file)

Add a Comment

Login with GitHub to post a comment