Commit e9d0a98
committed
fix(fe): links in new window (#2992)
### TL;DR
Added `target="_blank"` and `rel="noopener noreferrer"` to Discord and GitHub links in the sidebar.
### What changed?
- Removed unused icon imports (`faCheck` and `faTriangleExclamation`)
- Added `target="_blank"` and `rel="noopener noreferrer"` attributes to the Discord and GitHub links in the sidebar to ensure they open in new tabs
### How to test?
1. Navigate to any page with the sidebar
2. Click on either the Discord or GitHub links
3. Verify that they open in a new tab instead of replacing the current page
### Why make this change?
This improves user experience by allowing users to visit external resources without losing their current context in the application. The `rel="noopener noreferrer"` attribute also adds security by preventing the new tab from accessing the window.opener property.1 parent 99d9295 commit e9d0a98
1 file changed
+12
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
4 | 3 | | |
5 | 4 | | |
6 | 5 | | |
7 | | - | |
8 | 6 | | |
9 | 7 | | |
10 | 8 | | |
| |||
197 | 195 | | |
198 | 196 | | |
199 | 197 | | |
200 | | - | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
201 | 205 | | |
202 | 206 | | |
203 | 207 | | |
| |||
210 | 214 | | |
211 | 215 | | |
212 | 216 | | |
213 | | - | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
214 | 222 | | |
215 | 223 | | |
216 | 224 | | |
| |||
0 commit comments