?
avatar C-Lodder
C-Lodder
6 Jul 2020

Steps to reproduce the issue

  1. Go to /administrator/index.php?option=com_installer&view=install
  2. Go to the Install from Web tab
  3. Click any extension

Actual result

There are 2 icons on the buttons.

screeny

avatar C-Lodder C-Lodder - open - 6 Jul 2020
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jul 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Jul 2020
avatar brianteeman
brianteeman - comment - 6 Jul 2020

Thats because they are links to external websites that open in a new window and we have a css rule that matches that

avatar C-Lodder
C-Lodder - comment - 6 Jul 2020

So multiple icons are expected?

avatar brianteeman
brianteeman - comment - 6 Jul 2020

i am just explaining wht it happens. Not commenting on if it is correct

avatar Quy Quy - change - 20 Jul 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-07-20 23:28:37
Closed_By Quy
avatar Quy Quy - close - 20 Jul 2020
avatar Quy Quy - change - 4 Sep 2020
Status Closed New
Closed_Date 2020-07-20 23:28:37
Closed_By Quy
avatar Quy Quy - reopen - 4 Sep 2020
avatar HLeithner HLeithner - change - 8 Jan 2021
Labels Added: ?
avatar HLeithner HLeithner - labeled - 8 Jan 2021
avatar Akashrajpoots14
Akashrajpoots14 - comment - 13 Mar 2021

Hi, Sir, I want to work on this issue

avatar sandramay0905
sandramay0905 - comment - 13 Mar 2021

@Akashrajpoots14 you can make a pull request without allowing by another.

avatar infograf768
infograf768 - comment - 13 Mar 2021

It is not possible to correct this directly in core.

avatar chmst
chmst - comment - 13 Mar 2021

I expect that this rule is not applied on buttons.

avatar brianteeman
brianteeman - comment - 13 Mar 2021

It is not possible to correct this directly in core.

Yes it is. It is just changing the css rule

avatar shahbazalam07
shahbazalam07 - comment - 13 Mar 2021

I want to work on this issue

avatar gaurav24072002
gaurav24072002 - comment - 16 Mar 2021

Hey ! I am interested to solve this issue but when I go to the required instruction everything looks fine to me . @C-Lodder Please see the screenshot:
gsoc1

If you want to say something else please can you define specifically...?
And I would be more then happy to solve the issue...

avatar himanshu007-creator
himanshu007-creator - comment - 16 Mar 2021

Hey ! I am interested to solve this issue but when I go to the required instruction everything looks fine to me . @C-Lodder Please see the screenshot:
gsoc1

If you want to say something else please can you define specifically...?
And I would be more then happy to solve the issue...

looks good @gaurav24072002 , have a look HERE. Working on this issue is simpler and it would be good to see you work on it?.
I was working on it previously but found the discussion above unresolved. Ask about the desired layout and send a PR . ^_~

avatar brianteeman
brianteeman - comment - 16 Mar 2021

image

avatar brianteeman
brianteeman - comment - 16 Mar 2021

Are you sure that you were looking at Joomla 4

avatar himanshu007-creator
himanshu007-creator - comment - 16 Mar 2021

In joomla 4 , 2 ICONS ARE VISIBLE.
doubt:-
Now we have to remove one of the icon or place the external link icon on the right side of text?

avatar drmenzelit
drmenzelit - comment - 16 Mar 2021

I would say, we don't need the icon for external link here

avatar brianteeman
brianteeman - comment - 16 Mar 2021

@drmenzelit Then you might as well remove the icon from all buttons and links. The icon is not there as decoration. It exists to fulfil an accessibility requirement to indicate if a link will open in a new browser window.

avatar chmst
chmst - comment - 16 Mar 2021

I suggest to remove the decorative buttons - then it looks like this: The links are external links, the green button is a button, not a link becaus it opens a message and does not address another site at once.

grafik

avatar brianteeman
brianteeman - comment - 16 Mar 2021

The green button is multi-purpose and it depends on the actual extension on what it is and what it does.

avatar chmst
chmst - comment - 16 Mar 2021

Ok, my suggestion is - let the green button as it is, but remove the deorative icons from link buttons, let only the generated external-link icons.
This is a very easy fix in the layout.

avatar drmenzelit
drmenzelit - comment - 16 Mar 2021

@drmenzelit Then you might as well remove the icon from all buttons and links. The icon is not there as decoration. It exists to fulfil an accessibility requirement to indicate if a link will open in a new browser window.

sorry, I forgot...

avatar infograf768
infograf768 - comment - 17 Mar 2021

As the code producing these supplementary icons is NOT in core, the only way I found was to override it in _icon.scss.

diff --git a/administrator/templates/atum/scss/blocks/_icons.scss b/administrator/templates/atum/scss/blocks/_icons.scss
index 122bb50..863add6 100644
--- a/administrator/templates/atum/scss/blocks/_icons.scss
+++ b/administrator/templates/atum/scss/blocks/_icons.scss
@@ -86,2 +86,10 @@
   fill: var(--white);
 }
+
+#jed-container .icon-download::before {
+  content: "";
+}
+
+#jed-container .icon-list::before {
+  content: "";
+}

Screen Shot 2021-03-17 at 10 27 59

Note:

Did not test modifying in .../build/media_source/plg_installer_webinstaller/scss
But it would be better to modify the code where it is produced.

avatar ReLater
ReLater - comment - 17 Mar 2021

Or move the external icon ::after and not ::before.

avatar infograf768
infograf768 - comment - 17 Mar 2021

Or move the external icon ::after and not ::before.

Why not, but we have no power to do it in core...

avatar NikitaEmberi
NikitaEmberi - comment - 17 Mar 2021

Hi, I want to work on this issue but not really understanding where exactly would it be feasible to change the code in order to reflect changes. I have been trying but stuck. Can someone please help me on this like where is it coming from? Thanks.

avatar gaurav24072002
gaurav24072002 - comment - 17 Mar 2021

I was getting the three button when I used Joomla 3.x version but when I installed Joomla 4 I only got the download button !!
gsoc2

can anyone help me if I am missing something ?

avatar drmenzelit
drmenzelit - comment - 17 Mar 2021

@gaurav24072002 the screenshot shows the page of BreezingForms in the Extensions Directory... what I you trying to do?

avatar NikitaEmberi
NikitaEmberi - comment - 17 Mar 2021

I removed .icon-download:before, .icon-list:before, .icon-share-alt classes . Now it looks like this:
image
Would this be fine?

avatar ashuydv
ashuydv - comment - 18 Mar 2021

Can I work on fixing this issue??

avatar gaurav24072002
gaurav24072002 - comment - 18 Mar 2021

I made a[target=_blank]:before as display: none; and it looked like this:
image

Now we don't have two icon . Is this fine ?

avatar infograf768
infograf768 - comment - 18 Mar 2021

@gaurav24072002

I think your solution is fine.

Adding in /administrator/templates/atum/scss/blocks/_icons.scss

#jed-container a[target=_blank]:before {
  content: '';
}

or

#jed-container a[target=_blank]:before {
  display: none;
}

Then running npm ci
would do the job.

EDIT: But, as it may be needed for a11y as @brianteeman says, if we want 2 icons, one on each side, then the changes may have to be done in the specific external app and not in core.

avatar gaurav24072002
gaurav24072002 - comment - 18 Mar 2021

@infograf768 Thanks for you help.. I am on it..

avatar infograf768
infograf768 - comment - 18 Mar 2021

Modifying in the external app to get 2 icons will also require some padding

Screen Shot 2021-03-18 at 11 11 11

Taking off the icon is maybe the simplest way there.

avatar rdeutz rdeutz - change - 18 Mar 2021
Labels Removed: ?
avatar rdeutz rdeutz - unlabeled - 18 Mar 2021
avatar drmenzelit
drmenzelit - comment - 18 Mar 2021

After some discussions in the maintainer team, we think the best way to solve this issue is to move the external link icon after the text of the button: decorative icon - text - external link... That is more or less standard in web.

avatar C-Lodder
C-Lodder - comment - 18 Mar 2021

lol

avatar gaurav24072002
gaurav24072002 - comment - 18 Mar 2021

But what I suggested also looked fine according to me but it's ok we can also move the external icon towards right of text . can you please assign this issue to me . as I am working on this issue for long and I know the code base of it.

avatar rdeutz
rdeutz - comment - 18 Mar 2021

@gaurav24072002 we don't use assiging so often, I assign issues/PR to not forget what I have to look at. You can make a PR and set it to draft that should be enough, if you then mention the issue in your PR people can see that you are working on it.

avatar gaurav24072002
gaurav24072002 - comment - 18 Mar 2021

ok sure ?

avatar gaurav24072002
gaurav24072002 - comment - 23 Mar 2021

Hey! can anyone please tell me where to find the SCSS file of extensions??

avatar rdeutz rdeutz - change - 24 Mar 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-03-24 23:17:47
Closed_By rdeutz
avatar rdeutz rdeutz - close - 24 Mar 2021
avatar rdeutz
rdeutz - comment - 24 Mar 2021

Closing because we have a PR #32851

Add a Comment

Login with GitHub to post a comment