E532 Add edit button to media item within a form by jacob-v-dam · Pull Request #3192 · forkcms/forkcms · GitHub
[go: up one dir, main page]

Skip to content
8000

Add edit button to media item within a form#3192

Merged
carakas merged 2 commits intoforkcms:masterfrom
jacob-v-dam:add-edit-to-media-form-widget
May 3, 2021
Merged

Add edit button to media item within a form#3192
carakas merged 2 commits intoforkcms:masterfrom
jacob-v-dam:add-edit-to-media-form-widget

Conversation

@jacob-v-dam
Copy link
Contributor
@jacob-v-dam jacob-v-dam commented Oct 19, 2020

It is nice to edit the title of a media item within a from. When
there are a large amount of files it is hard to find the right file.
With this addition it is possible to click the edit pencil and change
the title.

Type

  • Enhancement

Resolves the following issues

Pull request description

With this function it would be able to change the title of a media item within a form. This means that the user doesn't have to search the whole library to find one item.

@jacob-v-dam jacob-v-dam requested a review from a team as a code owner October 19, 2020 13:08
@jacob-v-dam jacob-v-dam force-pushed the add-edit-to-media-form-widget branch 3 times, most recently from 45632ef to 2165b7b Compare October 19, 2020 13:26
@carakas carakas added this to the 5.10.0 milestone Oct 25, 2020
@codecov
Copy link
codecov bot commented Nov 22, 2020

Codecov Report

Merging #3192 (7d68e32) into master (38f5836) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3192      +/-   ##
============================================
- Coverage     27.79%   27.77%   -0.03%     
- Complexity     8010     8017       +7     
============================================
  Files           568      569       +1     
  Lines         30564    30591      +27     
============================================
  Hits           8496     8496              
- Misses        22068    22095      +27     
Flag Coverage Δ Complexity Δ
functional 23.70% <0.00%> (-0.03%) 0.00 <7.00> (ø)
installer 3.86% <0.00%> (-0.01%) 0.00 <7.00> (ø)
unit 7.62% <0.00%> (-0.01%) 0.00 <7.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...d/Modules/MediaLibrary/Ajax/MediaItemEditTitle.php 0.00% <0.00%> (ø) 7.00 <7.00> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38f5836...7d68e32. Read the comment docs.

@carakas
Copy link
Member
carakas commented Nov 22, 2020

@jacob-v-dam can you look at the feedback of the failing yarn test?

Jacob van Dam added 2 commits February 9, 2021 22:35
It is nice to edit the title from a media item within a from. When a
there are a large amount of file it is hard to find the right file.
With this addition it is possible to click the edit pencil to change
the title.
@jacob-v-dam jacob-v-dam force-pushed the add-edit-to-media-form-widget branch from a8fabd5 to 7d68e32 Compare February 9, 2021 21:47
@jacob-v-dam
Copy link
Contributor Author
jacob-v-dam commented Feb 9, 2021

Yarn tests are fixed. I see that codecov is failing right now, but only the storage manager is tested.

@carakas carakas merged commit 82fa42f into forkcms:master May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0