[4.1] Adaptive Image Plugin : New Media Manager #21775

geetanshu-m · 2018-08-21T02:00:51Z

Summary of Changes

The main objective of this extension is to preserve the important message associated with the image, by providing the user a way to add the focus area for the image. Images are being made responsive with the help of this extension.

Features

Provided a focus editor for taking in the focus information from the author, in a more intuitive manner.
Auto-Cropping of the images according to the viewport of the client device.
The cropping is done around the focus area selected by the author.
Author can select particular widths for which the images need to be cropped.
Author can select different focus for different devices.
Author can also remove all the resized images and the previous set focus points for a particular image from the focus editor.
If an author deletes the original image from the media manager, its resized image and the focus points would be deleted too.

Demo Video
Documentation

~~Note : This features can not be tested now because of this issue.~~

~~But after this got resolved one can test this feature by following these simple steps~~

All Good, feature is ready to be tested.

Testing Instructions

After pulling this PR to the local repository, Fresh install Joomla (Just delete configuration.php) or you can discover and enable the following plugin in the system through plugin extension manager:
- Focus (Media-Action Plugin)
- adaptiveimage (Content Plugin)
After the setup go to Media-Manager --> Edit any Image --> Go to focus tab.
Here you need to select a focus area for the image .
After selecting focus area Just hit Save and Close.
Finally, add that image to the article.
Then, Go to that article in the front-end and try changing the window size for the article.

Expected result

The cropped and smaller images around the focus area would be shown for smaller view ports.

More Information about the features and instruction to use them for this plugin is given in documentation here

* plugin added * moved responsifyjs to vendor * added quality field and it's string * modified smartcrop.js file * modified smartcrop.js file * Register smart crop as an event * added focus editor * Adding Responsify JS to template * solved deactivate not found error * removeing responsifyJS from the template * added responsifyJS * a very silly mistake * testing * Added Adaptive Image Controller * adaptive image content plugin * cleanup * added responsify script to html * Final setFocus method * completed getFocus method * demo focus file * functional adaptive image * cleanups * some minor modifications * removeing adaptive image controller from com_media * added documentation for adaptive image content plugin * cleanup * responsifying only the images haveing focus point * removeing previous styles on <img> * added interface * added focus.json in gitignore * updated adaptive image controller * updated .gitignore * removed focus.json * resolved 404 error * updated the changes * Seperated AdaptiveImage Interface * removed phpcs errors from adaptiveimage.php * updated phpcs errors of AdaptiveImageController.php * removed phpcs error from AdaptiveImageInterface.php * more fixes * Save and close button working for smart-crop * moved Adaptive Image controller from com_content to com_media * Separated Focus Store Class * removed break statement * update responsifyJS * Rename FocusStore to JSONFocusStore * updates * Renameing the folder * Updated the usage of interfaces * removed if else * using input for takeing values * update content plugin * update responsifyJS * updated master * removed adaptiveimage from administrator * removed performTask * makeing imgPath consistent * using file_put_content and file_read_content * phpcs * Accurate Focus points * Retriving focus points in focus editor * update responsifyJS * build * Jquery to vanillaJS * maintaining aspect ratio * removed responsifyJS * Adding plugin to the installation script * Class for System Cropping * System cropping working through URL * Default width * Crop Images on Save and Close * cache image name is original image name * Retriveing cache images in front-end * <picture> Element Working * removeing data-focus points * updateing gitignore * moved controller * phpcs * change ordering of images in <picture> * removed reponsifyJS from package.json * Removed unwanted variables * cleanup * for extra large images

* Adding more Fields to the front-end * Sending and receiveing Widths through ajax * updateing controller for requested widths * Added Multiple Focus Areas for diffrent widths * Added translatable strings * resolveing merge conflicts * phpcs * phpcs * added new lines * updated ajax for getFocusPoints * build * updateing ajax for all and build * removed data fields * codestyle * fix * codestyle fix * fix maxWidth and maxHeight * fix space * fix

* Adding more Fields to the front-end * Sending and receiveing Widths through ajax * updateing controller for requested widths * Added Multiple Focus Areas for diffrent widths * Added translatable strings * resolveing merge conflicts * phpcs * phpcs * added new lines * updated ajax for getFocusPoints * build * updateing ajax for all and build * removed data fields * Added Base for custom width * Fetching the parameters * Added custom widths to the front end * build * adjustng widths in the controller * pulled upstream * phpcs * fix * fix * Update smartcrop.xml * codestyle * removed break statement * fix * build * lines before control statement * fix * fix

@todo

* Adding more Fields to the front-end * Sending and receiveing Widths through ajax * updateing controller for requested widths * Added Multiple Focus Areas for diffrent widths * Added translatable strings * resolveing merge conflicts * phpcs * phpcs * added new lines * updated ajax for getFocusPoints * build * updateing ajax for all and build * removed data fields * Added Base for custom width * Fetching the parameters * Added custom widths to the front end * build * adjustng widths in the controller * pulled upstream * phpcs * fix * fix * Update smartcrop.xml * codestyle * removed break statement * fix * build * lines before control statement * phase 1 * fix * updated interface * changed the cache folder location * delete focus method * method for deleteing resized images * calling methods through controller * fixes * adding new location for resized images in gitignore * final working methods * deleteing the resized images&focus points if original image is not found * adding <picture> only if focus points are set * adding comments * fix * removed Quality field * delete button working * changeing position * fix * phpcs * phpcs * resolved multiple delete button addition error * changeing to POST * removeing else * @todo * >0

@todo

* Adding more Fields to the front-end * Sending and receiveing Widths through ajax * updateing controller for requested widths * Added Multiple Focus Areas for diffrent widths * Added translatable strings * resolveing merge conflicts * phpcs * phpcs * added new lines * updated ajax for getFocusPoints * build * updateing ajax for all and build * removed data fields * Added Base for custom width * Fetching the parameters * Added custom widths to the front end * build * adjustng widths in the controller * pulled upstream * phpcs * fix * fix * Update smartcrop.xml * codestyle * removed break statement * fix * build * lines before control statement * phase 1 * fix * updated interface * changed the cache folder location * delete focus method * method for deleteing resized images * calling methods through controller * fixes * adding new location for resized images in gitignore * final working methods * deleteing the resized images&focus points if original image is not found * adding <picture> only if focus points are set * adding comments * fix * removed Quality field * delete button working * changeing position * fix * phpcs * phpcs * resolved multiple delete button addition error * changeing to POST * removeing else * @todo * >0 * renamed smart crop to the focus * removed responsive-image.js * added gitignore * fix * Delete focus.json * adding focus.json to data/focus.json * cleanup * touch() issue solved * phpcs * alignment

added params to the installation script

@wilsonge

* [4.0] Pagination [a11y] admin (joomla#21293) * [4.0] Pagination [a11y] * Update default.php * Update modal.php * Update modal.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * Update default.php * tabs * Remove registering form paths for paths that no longer exist (joomla#21279) * Remove random number from code comment * [4.0] Separate sources from the distributable assets (joomla#21328) * I like to move it, move it 🕺 * Fix overflow of submenu despite being closed (joomla#21316) * [4.0] Fixed the translation of the view type in menuitem (joomla#21324) * Fixed the path to the XML-File of a layout in a view of a component. * Add the path to layout schema for Joomla! 3 component * fixed coding standard * fixed more coding standard * fixed the path to layout shema for old Joomla! version * Codestyle * 4.0 check for valid session before messing with ini (joomla#21337) * Added checks for an active PHP session before trying to change ini values. * Revert "Added checks for an active PHP session before trying to change ini values." This reverts commit 54aab95. * Added checks for an active PHP session before trying to change ini values. * Changed check to isActive() as suggested by @wilsonge . * Remove tabs/blank line (joomla#21346) * Update the cli nom commands (joomla#21350) * Ooops * Update package.json * Update compile-es6.js * remove the automated part here * Adjusted the batch-language.js to hide the batch move/copy options by default. (joomla#21340) * [4.0] Article batch operation fixes (joomla#21325) * some changes in ArticleModel * Undefined class warnings resolved * Commented out code removed * few style changes * few more style changes * Removed the function which caused an error because it wasn't declared anywhere * [4.0] Improving menu item workflow (joomla#21357) * created new branch * formatted code * fixed modal id variable loaded language file for modal title * formatted code and ajusted comments * fixed javascript error due to modal not closing * code style * code style * Codestyle * [4.0] CSS codestyle offline.css (joomla#21379) * Sans Serif to sans-serif + newline * cs * New version of Joomla * [4.0] update the custom elements (joomla#21204) * update joomla-custom-elements + polyfills * REMOVE * [4.0] Fixed the parent item dropdown (joomla#21348) * Fixed the parent item dropdown * fixed code style * Fix 2nd level menu (atum template) not showing (joomla#21367) * fixed button description to language string (joomla#21387) * [4.0] RTL frontend login (joomla#21402) * [4.0] Fix joomla#21357 when multilanguage is implemented (joomla#21391) * [4.0] Fix: Associations tab should display only one Notice when item edited is set to ALL languages * correcting my local branch * [4.0] Fix 21357 when multilang is implemented * [4.0] ES6 conversion of select-colour.js (joomla#21406) * select-colour * make sure we always compare numbers * correction: event name * [40] ES6 batch language script and move it to the right place (joomla#21405) * batch language * correction: event name * [4.0] Accessing property without checking the object state causing Joomla installer non-responsive (joomla#21413) * Bug fix for joomla#21412 After installing Joomla, go to customization and select language say Hindi for installation. If the language cannot be installed, the server will not return any value, and the value is null. But the below code is accessing its properties without checking the data is null or not. * Update LanguagesModel.php * Remove extra space * Remove extra tabs * [4.0] ES6 multiselect.js (joomla#21409) * multiselect * hound * this needs to be in a higher element * drop this * [4.0] Simplifying lang load (joomla#21415) * [4.0] Update readme.md (joomla#21392) Remove reference to jenkins * es6 please (joomla#21341) * Better error message for MySQL 8 workaround (joomla#21339) * Better error message for MySQL 8 workaround * change error message * error message typo in en-US * Fix typo in en-GB * [4.0] DRY for warning pages (joomla#21331) * DRY * Update package.json * Update restore_finalisation.php (joomla#21332) * [4.0] [a11y] Pagination site (joomla#21306) * [4.0] [a11y] Pagination * drone * cs * template css etc * woof woof * jtext * correct scss to highlight current * hound * fix link styles * hound * conflicts * revert joomla#18533 and move back to the stable state (joomla#18558) * [4.0] ES6 please, validate.js (joomla#21417) * validate * missing class * [4.0] Fix: Associations tab should display only one Notice when item edited is set to ALL languages (joomla#21362) * [4.0] Fix: Associations tab should display only one Notice when item edited is set to ALL languages * correcting my local branch * [4.0] Fix: Associations tab should display only one Notice when item edited is set to ALL languages * [4.0]namespace administrator components com_languages (joomla#21058) * com_languages * Update edit.php * Update StringsModel.php * Update OverridesModel.php * [4.0]namespace administrator components com_installer (joomla#21056) * com_installer * Update InstallModel.php * Update UpdateModel.php * jupdatecheck (joomla#21421) * bug in the regex (joomla#21427) * [4.0] Split the configuration of the session service on a per-application basis, restructure session GC command (joomla#19594) * Split the configuration of the session service on a per-application basis, restructure session GC command * Make a session factory, move handler creation into it * Expand comments on session service key aliasing * Apparently this is our PHPCS style? * Change to a less than readable variant of the signature because our PHPCS rules suck * Fix container reference * PHPCS fixes * Inject Session class options into factory, validate option presence * [4.0] Cleanup error handling in the installer on failed DB Connections (joomla#20334) * Cleanup of messages in installation error cases * Implement feedback * Code cleanup * Remove invalid character from cache namespace declaration * com_wrapper ES6 (joomla#21430) * Fix namespace code style of com_admin * Fix namespace code style of com_associations * [4.0] ES6 com_tags (joomla#21432) * com_tags * mute the dog * [4.0] ES6 com_languages overrider (joomla#21434) * overrider * woof * grrr * [4.0] ES6 please, tinyMCE (joomla#21428) * tinyMCE * mute the dog * remove the es5 files * No console errors on the BS and tiny sourcemaps * [4.0] ES6 please, sampledata.js (joomla#21422) * sampledata * mute the dog * [4.0] Change com_cpanel to services (joomla#21419) * Change com_cpanel to services * dispatcher * fields (joomla#21440) * [4.0] ES6 please, com newsfeed (joomla#21433) * com_newsfeed * woof * woof ? * [4.0] Install discover 500 (joomla#21443) Wrong namespace change - my bad * com_mailTo (joomla#21442) * [4.0] ES6 please, com_finder (joomla#21438) * finder * hound * These were supposed to be ES6 * woof * [4.0] Update build.js (joomla#21446) The installation template was missing from the watch and build scrippts * [4.0]Archive view (joomla#21449) Fix the missing endif to the title * Change content history to services (joomla#21418) * use the class keyword * plg_stats (joomla#21455) * [4.0] ES6 please, com_contacts (joomla#21453) * com_contacts * cleanup * [4.0] ES6 Please com_contenthistory (joomla#21454) * history * woof * Installer library - use parameterized queries (joomla#21459) * [4.0] Installer library qn/q (joomla#21461) Change the usage of q and qn to quote and quoteName in this recently merged code * [4.0] ES6 please: core+keepLive (joomla#21404) * core+keepLive * Ooops, missing things here * CS * fixes * [4.0] Password eye button (joomla#21462) Changed the class in the layout and the js * [4.0] ES6 please, com media edit (joomla#21444) * media edit * edit * woof * oops * hmm * really? * [4.0] Admin Pagination [a11y] (joomla#21424) * [4.0] Admin Pagination [a11y] Similar to joomla#21306 but for the admin The aim of this PR is to make the pagination seen in list views in the front end accessible - there are no visual changes - Wrap the pagination in a nav and add a role (needed for IE) and an aria-label - Ensure font icons have aria-hidden=true - Add aria-current to the selected page - Add aria-title to the active links eg Go to page 1 Note 1 that the disabled links will not be announced to a screen reader which is the intended behaviour Note 2 the list is an unordered list which appears to be the standard as otherwise the links would be announced as Link 1 go to page 2 etc which is nonsense ### Changes **administrator\templates\atum\html\pagination.php** removed - we dont need an override **layouts\joomla\pagination\links.php** Made similar to list and set showlimitbox to false (looks like legacy code to me) ** administrator\templates\atum\scss\blocks\_layout.css new file with just the styling used in the front end ### Testing Should be testable with patchtester but make sure that the template override has been deleted * cs * update joomla-browser [CLEAR CACHE] * move rebuild cache step before the tests * [4.0] ES6 please, com menu (joomla#21463) * com_menus * woof * Check that custom session path exists, create if not, if can't create then use default (joomla#19431) * Destroy session on deleting installation folder (joomla#20426) * xml codestyle * [a11y] Joomla logo is a decorative image only so should not have an alt description * Remove duplicate system-message-container div - important so we have unique id * fonrawesome is include in the template.css by the sccs - no need to load it again * update code comment it is super user not supr admin etc * use quote and quotename not q and qn * namespace - JText * Namespace JHtml * Help Docs link - add a link to the docs site and meaningful text for screenreaders to the "lightbulb" [a11y] * change section role=main to just <main> * [a11y] Add h1 to the page * You can not use a custom element to display the noscript message - obviously that's not going to work because the custom element is javascript - inception * [a11y] Move message about installing languages to be a caption [a11y] add scope to languages table * [a11y] contrast on language version number * [a11y] Section headings not nested correctly h1->h3 now h1->h2 * Namespace JFile * Namespace Jfolder * target blank on mysql8 link to docs * [a11y] footer link color and hover fixes joomla#21359 * Validate name of database fixes joomla#15583 * remove all references to switcher - not being used and it breaks scss compile * template.css after running npm build:css [note this hadn't been run for a while as it was missing from the build scripts] * forgot to save * quoteName * hound * missing semicolon in original code * aria-hidden * quote issue reported by @Quy * aria-hidden * fix regex - thank @ggppdk * regex again * [4.0] Remove Exception-catching from JMail (joomla#21320) * Updated the Mail.php file to the changes in PR joomla#12179 * Added Exception Handling for the Mails that get send after sending private messages in the Backend * Update Mass-Mail Function to handle the Exceptions from the JMail class * Update the update notification plugin to handle the new Exceptions from the JMail class * Update the Contact-Module to handle the exceptions thrown by the JMail class * Update joomla.php When you create a user and the creation email cant be sent, the thrown exception from the JMail class is handled * Update DisplayController.php When you try to forward an article via email and it fails the exception os caught and handled * Update RemindModel.php When you try to send a reminder email because you forgot your username and the mail can not be sent the exception is caught and handled. * Update ResetModel.php When you try to send a reset email because you forgot your password and the mail can not be sent the exception is caught and handled. * Update RegistrationModel.php When you try to try to register a new User account in the frontend, and the registration-mail can not be sent the exception is caught and handled. * Updated multiple files to better handle the new exceptions thrown by the JMail class and still display the correct messages. * Updated MessageModel.php to better handle the Exceptions thrown in the JMail class. Added new Error Message to translation file. * Removed unnecessary redirect from DisplayController.php * Improved Exception handling in the joomla.php that is used when creating users in the backend. * Updated translation file * Fixed a code-style issue * fixed a twice used class * [4.0] Feature/New Publishing Workflow (joomla#20381) Thanks @bembelimen @Didldu-Florian @puneet0191 @priiish @brianteeman and @chmst * [4.0] Constant usage (joomla#18504) * CONSTANT usage * Revert change in one case, as it might be useful to have the extra info there. * U Can't Touch This! * Fix conflicts and remove some replacements due to possible b/c break * By default categories use the default workflow * [4.0] [a11y] com_users icons (joomla#21468) While testing joomla#21441 (which is not related to this pr) I spotted that we missed three icons that should have aria-hidden applied to them The three icons can be seen in the admin when listing users - icon to add a note for the user. - icon to filter the notes list. You will need to create some notes first - number of notes for the user. You will need to create some notes first * [4.0] ES6 please, com modules (joomla#21465) * com_modules * woof * [4.0] ES6 please, searchtools (joomla#21467) * searchtools * haha * Fix namespace code style of com_banners * Fix namespace code style of com_cache * Fix namespace code style of com_categories * Fix namespace code style of com_checkin * Fix namespace code style of com_config * Fix namespace code style of com_contact * Fix namespace code style of com_content * Fix namespace code style of com_contenthistory * Fix namespace code style of com_cpanel * Fix namespace code style of com_csp * Fix namespace code style of com_fields * [4.0] workflow namespace (joomla#21471) * [4.0] workflow namepsace Quick pr to namespaace jtext and jtml * oops * [4.0] Workflows - fix xml codestyle (joomla#21470) fix various codestyle issues * Add missing doc block * [4.0] MVC mot Mvc (joomla#21478) * [4.0] MVC mot Mvc Codestyle * oops * [4.0] Workflow - too (joomla#21480) * [4.0] Workflow - too many In english it is "too many" when you refer to more than. You use "to" when you refer to travelling from A to B * Revert "[4.0] Workflow - too many" This reverts commit cd2058e. * too many * [4.0] Use the new hasField() in JTable (joomla#21479) * [4.0] Move JTable property_exists to hasField method Move JTable property_exists to hasField method * asset table * CoreContent * Extension * Nested * [4.0] Workflow namespace 2 (joomla#21484) * jroute * Jsession * jfactory * Jlayouthelper * jtext * Bump Composer dependencies (joomla#21490) * [4.0] Workflow [a11y] (joomla#21488) Remove th and scope=col from the checkall cell Add th and scope=row to the title cell * [4.0] Installer - precheck css (joomla#21482) * [4.0] Installer - precheck css PR for joomla#20773 ### Before ### After ### Testing You will probably need to force an error on your system or to edit the file installation\src\Model\ChecksModel.php and change some of the lines with option->state to $option->state = count($available); * woof * [4.0] Workflow unused variable (joomla#21486) Remove unused variable - probably remnant from removing the tfoot * [4.0] Workflows commented (joomla#21483) Remove 1 block of commented out code Expose 1 block of commented out code * [4.0] Workflow remove copy/paste code (joomla#21489) Can only assume that this id is from some other code that was copy pasted here without thought as it makes no sense to have this id * Fix namespace imports in workflow Controller * Codestyle fix * [4.0] Workflows language (joomla#21493) * [4.0] Workflows language en-gb and codestyle * oops * Remove unused import * Fix namespace * [4.0] Use the new hasField() (joomla#21494) * [4.0] Use the new hasField() * use the new hasField() * use the new hasField() * Codestyle and use whereIn from db driver * [4.0] c/p error in com_installer namespace commit (joomla#21492) JVERSION is a string for translation. * I need to setup my IDE * Array -> integer * Use namespaced class * Cleanup return type * Fix code comments * use function not property * Fix docblocks * [4.0] Fix npm script on windows (joomla#21505) * Fix compile-es6 script on Windows machines. * Adding back .min.js * [4.0] Fix workflow state submenu (joomla#21508) * Fix redirect for state checkin * Clean up * Fix broken multilingual handling (joomla#21512) * Fix workflows.xml (joomla#21511) * Fix namespace code style of com_finder * Fix namespace code style of com_installer * Fix namespace code style of com_joomlaupdate * Fix namespace code style of com_languages * Fix namespace code style of com_login * Fix namespace code style of com_menus * Fix namespace code style of com_messages * Fix namespace code style of com_modules * Fix namespace code style of com_newsfeeds * Fix namespace code style of com_plugins * Fix namespace code style of com_postinstall * Fix namespace code style of com_redirect * Fix namespace code style of com_search * Fix namespace code style of com_tags * Fix namespace code style of com_templates * Fix namespace code style of com_users * com_templates ES6 (joomla#21431) * Merge artcle title, alias, category to one column (joomla#21503) * [WIP][4.0] - workwflow mysql schema (joomla#21499) * [4.0] - workwflow mysql schema [wip] * extension field consistent make the extension field consistent with the current schema ie varchar(50) * Fix joomla#21517 * Namespaced class/docblock * Codestyle * Improve docblocks * Use namespaced code * Use namespaced table class * use a newer version of joomla-browser with higer timeout * [4.0] Fields redo, switcher (joomla#21227) * do not recreate child elements on node move * Change span to fieldset to properly adapt focus * label for sould targeting the fieldset * one more try * again * aria-labeledby * Remove wrongly committed file * [4.0] layout missing from joomla#21424 (joomla#21561) * [4.0] layout missing from joomla#21424 * hound * [4.0] State to stage (joomla#21544) * Change "state" to "stage" * article-model * Change "state" to "stage" * Spelling correction controler/controller * Fix typos * Revert renaming of workflowstate * Revert renaming of workflowstate * Fix Language-key and String, rename the e-mail param, revert change of the classname * Fix typos * Some more state-stage changes * Email without hyphen * Language string copied * Fix errors in renaming state to stage for workflow (joomla#21563) * Fix errors in renaming state to stage for workflow * Fix table * [4.0] Use the new hasField() (joomla#21496) * [4.0] Use the new hasField() * use hasField() * use the new hasField() * use the new hasField() * use hasField() * oops * [4.0] FIX searchtools (joomla#21562) * Clean up and alpha-order imports (joomla#21577) Signed-off-by: Roland Dalmulder <contact@rolandd.com> merged with trust in phpstorm * Add back package lock file missing * Fix joomla#21177 * [4.0] Revert helper (joomla#21523) * Remove readded core.js * Remove outdated ContentHelper::filterText method * Remove outdated ContentHelper::getContexts method * Remove outdated ContentHelper::validateSection method * Update countTagItems * Readd initial archive condition for further use * Remove blank lines * Align "=" * Revert line * [4.0] Install database name validation (joomla#21583) Correct the regex on the database name to prevent _ as the first character * [4.0] Installation error message (joomla#21572) Make the error messages on field validation red * [4.0] ES6 please show on (joomla#21568) * showon * woof * 🐶 * Fix joomla#21304 Fix conflicting class import * Lowercase for consistency * [4.0] It was a diffenent factory (joomla#21587) * I was a diffenent factory * lowercase keyword * [4.0] Normalise display of searchtools specific fields for menu items (joomla#21581) and multilingual associations * added params to the installation script * update to 4.8.2 remove temp patch (joomla#21588) * [4.0] State to stage fixes (joomla#21566) * [4.0] Stage (joomla#21589) * [4.0] Stage Quick PR to make the column header "From..." match the column header "To..." and the associated message * cs * [4.0][Workflow] Lowercase Workflowstage JHtml and Field (joomla#21597) * Rename Workflow_S_tageField * Rename JHtmlWorkflowStage to JHtmlWorkflowstage * [4.0] Empty Trash (joomla#21602) Makes the empty-trash button red just as the delete button is * [4.0] Workflow copy paste (joomla#21607) Fix some copy paste errors in the doc blocks * [4.0] Webcomponent tab.js typo (joomla#21606) oriendation ==> orientation * [4.0] Moved style to the CSS file (backend - system template) (joomla#21569) * Moved style to the CSS file * Move style to the CSS file * Removing trailing spaces Thanks @Quy * [4.0] [WORKFLOW] Modifying Toolbar constant (joomla#21591) * [4.0] [a11y] Modal button colour (joomla#21605) Make sure the secondary button passes the color contrast check ### Before ### After (Does not need npm to test) * remove redundant code (joomla#21609) * [4.0] State to Stage (joomla#21571) * State to Stage * Update en-GB.plg_content_joomla.ini * Update joomla.xml * Update joomla.php * [4.0] - Workflow mysql/postgresql schema (joomla#21520) * [WIP][4.0] - workwflow mysql schema -idx prefix - removed ENUM field type * postgresql schema update workflow * fix mysql install 4 workflow * workflow mysql install review * now() * lost hypen * workflow postgresql schema * the state alter * the state alter * state2stage * state2stage * mysql 5.5 * mysql 5.5 * mysql 5.5 consistency * mysql 5.5 consistency * missed quote * remove duplicated tables * revert state change on #__content * revert state field change * revert start field change * revert start field change * Self contain webcomponent SCSS (joomla#21615) * [4.0] Rules (joomla#21616) Obvious error missing - Non functional change * Enable detectDebug in HTMLHelper::webcomponent by default (joomla#21617) * Fix namespace code style of mod_feed * Fix namespace code style of mod_logged * Fix namespace code style of mod_login * Fix namespace code style of mod_menu * Fix namespace code style of mod_popular * Fix namespace code style of mod_version * Fix namespace code style of com_banners * Fix namespace code style of com_contact * Fix namespace code style of com_content * Fix namespace code style of com_contenthistory * Fix namespace code style of com_fields * Fix namespace code style of com_finder * Fix namespace code style of com_mailto * Fix namespace code style of com_modules * Fix namespace code style of com_newsfeeds * Fix namespace code style of com_search * Fix namespace code style of com_tags * Fix namespace code style of com_users * Fix namespace code style of com_wrapper * Fix namespace code style of mod_banners * [4.0] allow jdoc:include type=message to load assets (joomla#21558) * dry * allow modules to enqueMessages * [4.0] - [workflow] fix undefined condition (joomla#21621) * [4.0] - [workflow] fix undefined condition * undefined condition * [4.0][workflow] - undefinded property on modal (joomla#21622) * Fix the imports in the archive model (joomla#21620) * [4.0] drop jQuery (joomla#21575) * Drop jQuery (again 😃) * typo * cs * remove redundant code * Update default.php * Update default.php * Update default.php * Update default.php * fix dos (joomla#21627) * [4.0][workflow] remove workflow from user menu (joomla#21626) * [4.0][workflow] remove workflow from user menu * remove content field item from user menu * [4.0] [a11y] Set login page title (joomla#21603) * [4.0] [a11y] Set login page title a11y rules says that every page should have a meaningfu title This PR adds the title "Administrator Login - sitename" to the title on the login page * save before commit * reverse * composer and npm install * [4.0] [workflow] - notice on edit plugin (joomla#21623) * [4.0] [workflow] - notice on edit plugin * conservative * don't kill empy lines * [4.0] com_workflow: Missing Value for "Count States" - fixes joomla#21634 (joomla#21635) * [4.0][workflow] - batch change stage (joomla#21625) * [4.0][worflow] - batch change stage * stage line 840 * build

This reverts commit 28dfe8a.

…ocus

kasvith · 2018-08-21T02:47:34Z

@laoneo @dneukirchen @dgrammatiko @ciar4n it would be great if you guys do some review

brianteeman · 2018-08-21T07:26:02Z

@kasvith kind of hard to test because as @proman24 days it cant be tested because of joomla-projects/media-manager-improvement#577

brianteeman · 2018-08-21T07:27:21Z

.gitignore

@@ -21,6 +21,7 @@
 /.php_cs
 /.php_cs.cache
 /media
+/plugins/media-action/focus/data/*


Why is this folder in the gitignore?

This folder stores the cache / cropped images.

I thought that might be the case - which means it is in the wrong place and should be a subfolder of /media

Sorry, wrongly said.... Cache / Cropped images are stored in /media/focus/*
Actually this folder contains the data-focus points associated with the images in JSON format. Currently we are using file-system to store the mapping of focus points and the images.

we still shouldnt be storing user created anything in this location

administrator/components/com_media/src/Controller/AdaptiveImageController.php

brianteeman · 2018-08-21T07:29:00Z

administrator/language/en-GB/en-GB.plg_content_adaptiveimage.ini

+; Joomla! Project
+; Copyright (C) 2005 - 2018 Open Source Matters. All rights reserved.
+; Note : All ini files need to be saved as UTF-8
+


did you mean to commit thiis without any strings?

brianteeman · 2018-08-21T07:29:55Z

administrator/language/en-GB/en-GB.plg_media-action_focus.sys.ini

@@ -0,0 +1,4 @@
+PLG_MEDIA-ACTION_FOCUS_FIELD-SET_DESCRIPTION="Resizes images to these widths, setting them to the default"


Please add a . to the end

brianteeman · 2018-08-21T07:30:49Z

administrator/language/en-GB/en-GB.plg_media-action_focus.sys.ini

+PLG_MEDIA-ACTION_FOCUS_FIELD-SET_DESCRIPTION="Resizes images to these widths, setting them to the default"
+PLG_MEDIA-ACTION_FOCUS_FIELD-SET_LABEL="Custom Widths"
+PLG_MEDIA-ACTION_FOCUS_TITLE="Media Action - Focus"
+PLG_MEDIA-ACTION_FOCUS_XML_DESCRIPTION="Plugin which automatically resize and crop the images in the responsive environment"


==> resizes
==> crops
in the ==> in a ??

Please alpha sort these use statements

brianteeman · 2018-08-21T07:31:34Z

libraries/src/AdaptiveImage/FocusStoreInterface.php

+<?php
+/**
+ * @package     Joomla
+ * @subpackage  com_media


package and subpackage need updating

package => Joomla
subpackage => AdaptiveImage ??

actually both can be removed

brianteeman · 2018-08-21T07:31:51Z

libraries/src/AdaptiveImage/FocusStoreInterface.php

+interface FocusStoreInterface
+{
+	/**
+	 * Pubic function for storeing the focus points


brianteeman · 2018-08-21T07:32:08Z

libraries/src/AdaptiveImage/FocusStoreInterface.php

+	public function getFocus($imgSrc, $width);
+
+	/**
+	 * Function for removeing the focus points for all widths


==>removing

brianteeman · 2018-08-21T07:32:23Z

libraries/src/AdaptiveImage/FocusStoreInterface.php

+	public function deleteFocus($imgSrc);
+
+	/**
+	 * Function for removeing all the associated resized images


==>removing

brianteeman · 2018-08-21T07:32:44Z

libraries/src/AdaptiveImage/JSONFocusStore.php

+<?php
+/**
+ * @package     Joomla
+ * @subpackage  com_media


package and subpackage need updating

brianteeman · 2018-08-21T07:33:12Z

libraries/src/AdaptiveImage/JSONFocusStore.php

+	 * @since   4.0.0
+	 */
+	protected $cacheDir =  "/media/focus";
+	/**


add a new line

libraries/src/AdaptiveImage/SmartCrop.php

media/com_media/js/edit-images.js

plugins/content/adaptiveimage/adaptiveimage.php

plugins/media-action/focus/focus.php

plugins/media-action/focus/focus.xml

bembelimen · 2022-01-20T03:37:12Z

@proman24 is there any interest to work through the feedback and make this PR mergable?

Co-authored-by: Phil E. Taylor <phil@phil-taylor.com>

HLeithner · 2022-06-27T13:09:37Z

This pull request has automatically rebased to 4.2-dev.

joomla-bot · 2022-06-27T21:48:57Z

This pull requests has been automatically converted to the PSR-12 coding standard.

alikon · 2022-07-13T07:42:02Z

closed, because has been taken over by another person #38264

geetanshu-m and others added 14 commits June 29, 2018 18:14

Cleaning path and Case Matching of Controller (joomla#10)

87dcea4

added params to the installation script (joomla#17)

6f7390c

added params to the installation script

Revert "New Media Manager Upstream Merge (joomla#18)" (joomla#19)

77609e7

This reverts commit 28dfe8a.

merge gsoc-18_adaptive_image

d3c7bdb

build

d0a9206

Merge branch '4.0-dev' of https://github.com/joomla/joomla-cms into f…

3ff7993

…ocus

Merge branch '4.0-dev' of https://github.com/joomla/joomla-cms into f…

a97ac81

…ocus

Merge branch '4.0-dev' of https://github.com/joomla/joomla-cms into f…

61f0c51

…ocus

geetanshu-m requested a review from brianteeman as a code owner August 21, 2018 02:00

joomla-cms-bot added Language Change This is for Translators PR-4.0-dev labels Aug 21, 2018

brianteeman reviewed Aug 21, 2018

View reviewed changes

administrator/components/com_media/src/Controller/AdaptiveImageController.php Show resolved Hide resolved

brianteeman reviewed Aug 21, 2018

View reviewed changes