diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 0000000..8066511 --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,35 @@ +# Pull Request + +Thanks for contributing to ElegantFin! Please review the contributor guidelines before submitting. + +## Type of Change + +- [ ] Bug fix +- [ ] New feature + +## Description + +Explain what you changed and why. Include reasoning and goals. + +## Screenshots + +If applicable, provide before/after images and explain what each shows. + +## Cross-platform Testing + +Describe how you verified your changes on different devices (desktop, mobile, TV, etc.). + +## Test Configuration + +- Jellyfin server version: +- Jellyfin client: +- Client browser name and version: +- Device: + +## Checklist + +- [ ] I performed a self-review of my own code +- [ ] I followed the style conventions (`em` units, minimal media queries). +- [ ] I avoided unnecessary use of `!important`. +- [ ] I commented my code where applicable. +- [ ] I tested my changes on multiple devices, layouts and viewport sizes. diff --git a/.github/PULL_REQUEST_TEMPLATE.yml b/.github/PULL_REQUEST_TEMPLATE.yml deleted file mode 100644 index 9952f26..0000000 --- a/.github/PULL_REQUEST_TEMPLATE.yml +++ /dev/null @@ -1,47 +0,0 @@ -name: Pull Request -description: Submit a contribution to ElegantFin -title: "[PR] " -labels: [] -assignees: [] - -body: - - type: markdown - attributes: - value: | - Please follow our [Contributor Guidelines](../CONTRIBUTING.md) and keep your PR focused on a single improvement or fix. - - type: textarea - id: description - attributes: - label: Description - description: Describe your changes and the reason behind them. - placeholder: What did you change and why? - validations: - required: true - - type: textarea - id: screenshots - attributes: - label: Screenshots - description: Add before/after images of any visual changes. - placeholder: Drag and drop or paste images here. - validations: - required: false - - type: textarea - id: testing - attributes: - label: Cross-platform Testing - description: Describe how you verified this works across devices (e.g., desktop, mobile, TV). - validations: - required: true - - type: checkboxes - id: checklist - attributes: - label: Checklist - options: - - label: I followed the style conventions (`em` units, minimal media queries). - required: true - - label: I avoided unnecessary use of `!important`. - required: true - - label: I commented my code where applicable. - required: true - - label: I tested my changes across viewports. - required: true \ No newline at end of file