From 2f9c28ec99a8c488c6791da30194a85e6c9aa4ce Mon Sep 17 00:00:00 2001 From: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com> Date: Tue, 24 Dec 2024 19:41:15 -0800 Subject: [PATCH] Fix formatting --- .github/pull_request_template.md | 19 ++++--- CONTRIBUTING.md | 54 ++++++++++--------- .../src/components/chat/ContextMenu.tsx | 2 +- 3 files changed, 43 insertions(+), 32 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index f559068252..a4dc71c669 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,23 +1,28 @@ ### Description + ### Type of Change + -- [ ] 🐛 Bug fix (non-breaking change which fixes an issue) -- [ ] ✨ New feature (non-breaking change which adds functionality) -- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) -- [ ] 📚 Documentation update +- [ ] 🐛 Bug fix (non-breaking change which fixes an issue) +- [ ] ✨ New feature (non-breaking change which adds functionality) +- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) +- [ ] 📚 Documentation update ### Pre-flight Checklist + -- [ ] Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs) -- [ ] Tests are passing (`npm test`) and code is formatted and linted (`npm run format && npm run lint`) -- [ ] I have reviewed [contributor guidelines](https://github.com/cline/cline/blob/main/CONTRIBUTING.md) +- [ ] Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs) +- [ ] Tests are passing (`npm test`) and code is formatted and linted (`npm run format && npm run lint`) +- [ ] I have reviewed [contributor guidelines](https://github.com/cline/cline/blob/main/CONTRIBUTING.md) ### Screenshots + ### Additional Notes + diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 1d853e2817..73a3acc0bf 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -25,42 +25,48 @@ If you're planning to work on a bigger feature, please create an issue first so Anyone can contribute code to Cline, but we ask that you follow these guidelines to ensure your contributions can be smoothly integrated: 1. **Keep Pull Requests Focused** - - Limit PRs to a single feature or bug fix - - Split larger changes into smaller, related PRs - - Try and limit the PR to a single commit + + - Limit PRs to a single feature or bug fix + - Split larger changes into smaller, related PRs + - Try and limit the PR to a single commit 2. **Code Quality** - - Run `npm run lint` to ensure code follows our style guidelines - - Run `npm run format` to format your code with Prettier - - Address any ESLint warnings or errors before submitting - - Follow TypeScript best practices and maintain type safety + + - Run `npm run lint` to ensure code follows our style guidelines + - Run `npm run format` to format your code with Prettier + - Address any ESLint warnings or errors before submitting + - Follow TypeScript best practices and maintain type safety 3. **Testing** - - Add tests for new features - - Run `npm test` to ensure all tests pass - - Update existing tests if your changes affect them - - Include both unit tests and integration tests where appropriate + + - Add tests for new features + - Run `npm test` to ensure all tests pass + - Update existing tests if your changes affect them + - Include both unit tests and integration tests where appropriate 4. **Documentation** - - Update documentation for any changed functionality - - Update the README if your changes affect usage or setup + + - Update documentation for any changed functionality + - Update the README if your changes affect usage or setup 5. **Commit Guidelines** - - Write clear, descriptive commit messages - - Use conventional commit format (e.g., "feat:", "fix:", "docs:") - - Reference relevant issues in commits using #issue-number + + - Write clear, descriptive commit messages + - Use conventional commit format (e.g., "feat:", "fix:", "docs:") + - Reference relevant issues in commits using #issue-number 6. **Before Submitting** - - Rebase your branch on the latest main - - Ensure your branch builds successfully - - Double-check all tests are passing - - Review your changes for any debugging code or console logs + + - Rebase your branch on the latest main + - Ensure your branch builds successfully + - Double-check all tests are passing + - Review your changes for any debugging code or console logs 7. **Pull Request Description** - - Clearly describe what your changes do - - Include steps to test the changes - - List any breaking changes - - Add screenshots for UI changes + - Clearly describe what your changes do + - Include steps to test the changes + - List any breaking changes + - Add screenshots for UI changes ## Contribution Agreement diff --git a/webview-ui/src/components/chat/ContextMenu.tsx b/webview-ui/src/components/chat/ContextMenu.tsx index d2dee98e22..7e56b332a5 100644 --- a/webview-ui/src/components/chat/ContextMenu.tsx +++ b/webview-ui/src/components/chat/ContextMenu.tsx @@ -129,7 +129,7 @@ const ContextMenu: React.FC = ({ style={{ padding: "8px 12px", cursor: isOptionSelectable(option) ? "pointer" : "default", - color: + color: index === selectedIndex && isOptionSelectable(option) ? "var(--vscode-quickInputList-focusForeground)" : "",