Skip to content

Conversation

@pregnantboy
Copy link
Contributor

Changes

  • Show import csv error messages
  • reduce font size on gsib

## Problem
Plumber looks zoomed in on GSIB

## Solution
By using the method in the previous PR. We simulate a zoom out by
setting the font-size to a smaller value than 16 (i.e. 14). This works
seemingly well since we use `rem` as much as possible.
…rt (#787)

## Changes
1. show error message when file is too large / invalid file type
2. change default dropzone background color

<img width="899" alt="image"
src="https://github.com/user-attachments/assets/aef903b6-934f-4932-8988-e4825840d7f0">
@pregnantboy pregnantboy requested a review from a team as a code owner November 7, 2024 13:16
@datadog-opengovsg
Copy link

Datadog Report

Branch report: develop-v2
Commit report: 952c5dc
Test service: plumber

✅ 0 Failed, 687 Passed, 0 Skipped, 2m 0.06s Total Time
➡️ Test Sessions change in coverage: 1 no change

Copy link
Contributor

@m0nggh m0nggh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok tested that it works on gsib: the difference in font size, lgtm

@pregnantboy pregnantboy merged commit e4d781f into production Nov 8, 2024
5 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants