WorryFree Computers   »   [go: up one dir, main page]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.1] Generic doc icon #42527

Merged
merged 5 commits into from
Dec 28, 2023
Merged

[5.1] Generic doc icon #42527

merged 5 commits into from
Dec 28, 2023

Conversation

brianteeman
Copy link
Contributor

Replaces the pdf icon used in media manager with a more generic doc icon as this is applied to a variety of doc types.

To test

Use one of the prebuilt packages for this PR and check to see that the icon used in media manager for any doc type is no longer a pdf icon.

Changing the icon to a specific one for each type of doc is beyond the scope of this PR

Pull Request for Issue #42518

Before

image

After

image
.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Replaces the pdf icon used in media manager with a more generic doc icon as this is applied to a variety of doc types.

### To test
Use one of the prebuilt packages for this PR and check to see that the icon used in media manager for any doc type is no longer a pdf icon.

Changing the icon to a specific one for each type of doc is beyond the scope of this PR
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.1-dev labels Dec 15, 2023
@ChristineWk
Copy link

The NPM Package (under Download) is not available.

@brianteeman
Copy link
Contributor Author

updating branch to see if it will trigger the downloads to be created

@dgrammatiko
Copy link
Contributor

@brianteeman #36723 has more icons

@brianteeman
Copy link
Contributor Author

but that is closed and as stated

Changing the icon to a specific one for each type of doc is beyond the scope of this PR

@richard67
Copy link
Member

The NPM Package (under Download) is not available.

@ChristineWk You can find the downloads here: https://artifacts.joomla.org/drone/joomla/joomla-cms/5.1-dev/42527/downloads/72301/ . Currently there is something broken with updating the links to these downloads on GitHub.

@ChristineWk
Copy link

@richard67
Thank you. Hv installed your download with relevant custom update server.

42527-1
42527-2
Can this be called successful? see 2nd picture (.doc)

@richard67
Copy link
Member

Can this be called successful? see 2nd picture (.doc)

@ChristineWk Not sure now and no time to check. @brianteeman Do you know why she gets the "No preview" icon for the 2nd case .doc?

@brianteeman
Copy link
Contributor Author
  1. did it show the pdf icon before?
  2. is .doc an allowed file type?

@ChristineWk
Copy link
  1. did it show the pdf icon before?
    sorry, I'm not sure

    1. is .doc an allowed file type?
      Yes

@brianteeman
Copy link
Contributor Author

An essential part of testing is to replicate the before scenario. As you can see in both my before and after screenshots both show the doc file with first a pdf and then a generic icon your test is not successful for some reason

@dautrich
Copy link
dautrich commented Dec 18, 2023

Screenshot 2023-12-18 at 14-10-38 Media - J5 1_Test - Administration
I have tested this item ✅ successfully on 7873ca8

Tested with document types PDF, TXT, DOCX, XLSX


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

@Quy
Copy link
Contributor
Quy commented Dec 19, 2023

I have tested this item ✅ successfully on 7873ca8


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

@Quy
Copy link
Contributor
Quy commented Dec 19, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 19, 2023
@Razzo1987 Razzo1987 merged commit 57f62a2 into joomla:5.1-dev Dec 28, 2023
2 checks passed
@Razzo1987
Copy link
Contributor

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Dec 28, 2023
@Quy Quy added this to the Joomla! 5.1.0 milestone Dec 28, 2023
@brianteeman brianteeman deleted the icon-doc branch December 28, 2023 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.1-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants