feat(drills): Update ice surface system with 5 views and full-res export #9
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feature/ice-surface-five-views"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
Updates the ice surface system in the drill designer with 5 named views and full-resolution export capabilities.
Changes
1. Ice Surface Upload Views (5 views)
Updated
views/admin_system_tools.phpandprocess_settings.phpto support 5 ice surface views:2. Full Resolution Export
Updated
js/drill_designer.jswith new export system:getExportDimensions()— Returns appropriate dimensions per view (9298×4320 for full)createExportCanvas(maxSize?)— Creates composite canvas with ice surface + drill objectsexportImage()— Downloads full-res PNG composite3. Drill Cards — Scale Without Losing Quality
Updated
views/drills_library.php:object-fit: containmaintains aspect ratio at any card size4. Detailed Drill View — Flex Box Display
Updated
views/view_drill.php:object-fit: contain5. CSS Aspect Ratios
Updated
css/drills.csswith new data-ice-view selectors for all 5 views including CenterIce (1:1 circle).Files Modified
views/admin_system_tools.php— Upload section with 5 viewsprocess_settings.php— Backend validation for new view namesjs/drill_designer.js— Export system + view mappingsjs/ice_canvas.js— View name mappingscss/drills.css— Aspect ratios for new viewsviews/drills_create.php— Ice view dropdownviews/drills_library.php— Card thumbnailsviews/view_drill.php— Flex box displaydrill_share.php— View name mappingspractice_plan_share.php— View name mappingsviews/practice_create.php— View name defaultsviews/practice_plans.php— View name defaultsprocess_drills.php— Allowed views + defaultsgenerate_rink_images.php— Image generation view namesView command line instructions
Manual merge helper
Use this merge commit message when completing the merge manually.
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.