[PR #3341] [MERGED] refactor: remove font sizes from the app #4350

New issue

Closed

opened 2026-03-17 01:53:34 +03:00 by kerem · 0 comments

kerem commented

2026-03-17 01:53:34 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3341
Author: @joeljstephen
Created: 9/11/2023
Status: ✅ Merged
Merged: 9/13/2023
Merged by: @AndrewBastin

Base: release/2023.8.1 ← Head: refactor/remove-font-sizes

📝 Commits (5)

c1bfb90 refactor: remove font size from settings
6a612b4 refactor: remove font size from themes
338fca2 refactor: remove font size from spotlight
578cbd6 refactor: remove default font size
b87c996 chore: clean up

📊 Changes

11 files changed (+70 additions, -399 deletions)

View changed files

📝 packages/hoppscotch-common/assets/scss/themes.scss (+15 -62)
📝 packages/hoppscotch-common/src/components/realtime/LogEntry.vue (+1 -5)
➖ packages/hoppscotch-common/src/components/smart/FontSizePicker.vue (+0 -71)
📝 packages/hoppscotch-common/src/layouts/default.vue (+3 -19)
📝 packages/hoppscotch-common/src/modules/theming.ts (+0 -14)
📝 packages/hoppscotch-common/src/newstore/settings.ts (+0 -6)
📝 packages/hoppscotch-common/src/pages/settings.vue (+1 -9)
📝 packages/hoppscotch-common/src/services/spotlight/searchers/settings.searcher.ts (+1 -67)
📝 packages/hoppscotch-sh-admin/assets/scss/themes.scss (+18 -65)
📝 packages/hoppscotch-sh-admin/index.html (+16 -19)
📝 packages/hoppscotch-ui/src/assets/scss/themes.scss (+15 -62)

📄 Description

Ticket

Closes HFE-215

Description

This PR removes the font sizes from the Hoppscotch app as it causes UI inconveniences and is difficult to maintain uniform styling across browsers with custom font sizes.

Objectives

Remove the font size option from the settings page
Remove the font size option from the spotlight search
Make small font size as the default font size for the app
Remove other font size specific styles from the app

Checks

My pull request adheres to the code style of this project
My code requires changes to the documentation
I have updated the documentation as required
All the tests have passed

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/3341 **Author:** [@joeljstephen](https://github.com/joeljstephen) **Created:** 9/11/2023 **Status:** ✅ Merged **Merged:** 9/13/2023 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2023.8.1` ← **Head:** `refactor/remove-font-sizes` --- ### 📝 Commits (5) - [`c1bfb90`](https://github.com/hoppscotch/hoppscotch/commit/c1bfb906dd58eeb278d5fbc9e228d6f4016bd65f) refactor: remove font size from settings - [`6a612b4`](https://github.com/hoppscotch/hoppscotch/commit/6a612b495924bd53ed390e439d2e50797956524b) refactor: remove font size from themes - [`338fca2`](https://github.com/hoppscotch/hoppscotch/commit/338fca2fdb19b6acc077dad8c3c9cb9110a837c6) refactor: remove font size from spotlight - [`578cbd6`](https://github.com/hoppscotch/hoppscotch/commit/578cbd672c6a3066ccb39cfbf7a4217cf7d9f74b) refactor: remove default font size - [`b87c996`](https://github.com/hoppscotch/hoppscotch/commit/b87c9966ab6bcc0a88d02cebec2f3b1cbef6af80) chore: clean up ### 📊 Changes **11 files changed** (+70 additions, -399 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-common/assets/scss/themes.scss` (+15 -62) 📝 `packages/hoppscotch-common/src/components/realtime/LogEntry.vue` (+1 -5) ➖ `packages/hoppscotch-common/src/components/smart/FontSizePicker.vue` (+0 -71) 📝 `packages/hoppscotch-common/src/layouts/default.vue` (+3 -19) 📝 `packages/hoppscotch-common/src/modules/theming.ts` (+0 -14) 📝 `packages/hoppscotch-common/src/newstore/settings.ts` (+0 -6) 📝 `packages/hoppscotch-common/src/pages/settings.vue` (+1 -9) 📝 `packages/hoppscotch-common/src/services/spotlight/searchers/settings.searcher.ts` (+1 -67) 📝 `packages/hoppscotch-sh-admin/assets/scss/themes.scss` (+18 -65) 📝 `packages/hoppscotch-sh-admin/index.html` (+16 -19) 📝 `packages/hoppscotch-ui/src/assets/scss/themes.scss` (+15 -62) </details> ### 📄 Description ### Ticket Closes HFE-215 ### Description This PR removes the font sizes from the Hoppscotch app as it causes UI inconveniences and is difficult to maintain uniform styling across browsers with custom font sizes. ### Objectives - [x] Remove the font size option from the settings page - [x] Remove the font size option from the spotlight search - [x] Make small font size as the default font size for the app - [x] Remove other font size specific styles from the app ### Checks  - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [x] All the tests have passed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>