Skip to content

Conversation

@Cryptic4Life
Copy link

@Cryptic4Life Cryptic4Life commented Nov 28, 2025

Closes #1707

Added a new "Hide sensitive info" toggle in the profile settings. When enabled, the average grade and ECTS bar in the academic path page are blurred by default. The user can tap the blurred section to reveal it, and users with the option disabled can also tap on the section to blur or unblur the information as needed.

Settings Blurred View
settings blur

Review checklist

  • Terms and conditions reflect the changes

View Changes

  • Description has screenshots of the UI changes.
  • Tested both in light and dark mode.
  • New text is both in portuguese (PT) and english (EN).
  • Works in different text zoom levels.
  • Works in different screen sizes.

Performance

  • No helper functions to return widgets are added. New widgets are created instead.
  • Used ListView.builder for Long Lists.
  • Controllers (TextEditingController, ...) are beeing disposed of in dispose() method.

@Cryptic4Life Cryptic4Life requested a review from a team November 28, 2025 11:21
@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60%. Comparing base (7d9c4e8) to head (0c0cf57).
⚠️ Report is 1 commits behind head on develop.

❌ Your project check has failed because the head coverage (60%) is below the target coverage (70%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           develop   #1745   +/-   ##
=======================================
  Coverage       60%     60%           
=======================================
  Files            2       2           
  Lines           81      81           
=======================================
  Hits            48      48           
  Misses          33      33           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@pedroafmonteiro pedroafmonteiro left a comment

Choose a reason for hiding this comment

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

Hi! This looks like a great implementation. Thanks for this!

However, the way it visually looks needs a little debating from the whole team which can be discussed better in the next meeting. If you can and have the time, you could maybe create different versions of how it looks based on your opinion and then we get back to you with the one we think it's the best in the meeting. What do you think?

Some people pointed out that they do not like the overall blur and prefer the widget to be completely hidden and show a button instead of it named "Show average" or something like that.

Others did not like how the blur was on the whole widget, maybe just on the numbers would be good.

Therefore, creating different versions would allow the team to get a better view on how this could be well implemented!

Thanks again!

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.

Make private and sensible information hidden with the option to be the default or not

3 participants