Few improvments as fixes #218

Merged
Llewellyn merged 2 commits from staging into master 5 years ago
Owner

Pull Request for Issue gh-212 gh-217, gh-216.

Summary of Changes

  • Added (Linked To) back in related areas, so we can see where what is linked.
  • Added ajax fields to the custom admin view.
  • Added some backward compatibility to stylesheets and scripts being added.
  • Fixed no access lang string.
  • Added the site CSS area to Joomla Component view.

Testing Instructions

  • Open admin_views, custom_admin_views, site_views, fields, dynamic_gets, libraries to see if the linked to values are correct.
  • See if the ajax fields in the custom admin works.
  • check the new compatibility check for style sheets and scripts are behaving correctly.
  • Is the language string for no access loading.
  • Add some css (to site) in the joomla_component area

Expected result

  • All linked relationships must show
  • Be able to add ajax to the custom admin view directly.
  • backward compatibility with Joomla 3.7 and lower.
  • able to add global css to front-end

Actual result

None of the expected results were possible before.

Pull Request for Issue gh-212 gh-217, gh-216. ### Summary of Changes - Added (Linked To) back in related areas, so we can see where what is linked. - Added ajax fields to the custom admin view. - Added some backward compatibility to stylesheets and scripts being added. - Fixed no access lang string. - Added the site CSS area to Joomla Component view. ### Testing Instructions - Open admin_views, custom_admin_views, site_views, fields, dynamic_gets, libraries to see if the linked to values are correct. - See if the ajax fields in the custom admin works. - check the new compatibility check for style sheets and scripts are behaving correctly. - Is the language string for no access loading. - Add some css (to site) in the joomla_component area ### Expected result - All linked relationships must show - Be able to add ajax to the custom admin view directly. - backward compatibility with Joomla 3.7 and lower. - able to add global css to front-end ### Actual result None of the expected results were possible before.
kevincam3 (Migrated from github.com) reviewed 5 years ago
Roline approved these changes 5 years ago
ro-ot (Migrated from github.com) approved these changes 5 years ago
ro-ot (Migrated from github.com) left a comment
Poster
Owner

Thanks for adding the linked to, that really helps.

Thanks for adding the **linked to**, that really helps.
Llewellyn added this to the Feature Requests project 8 months ago

Reviewers

Roline approved these changes 5 years ago
The pull request has been merged as 3de31b1c1a.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b staging master
git pull origin staging

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff staging
git push origin master
Sign in to join this conversation.
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: joomla/Component-Builder#218
Loading…
There is no content yet.