Skip to content

Conversation

@subsymbolic
Copy link
Contributor

@subsymbolic subsymbolic commented Feb 27, 2018

Asana Issue URL: https://app.asana.com/0/414730916066338/574830763339677

Description

Fixes incorrect padding for the enhanced grade in the scorecard activity

Steps to Test this PR:

  1. Visit cnn.com
  2. Open the Privacy Dashboard
  3. Tap on the Scorecard
  4. Note that the "enhanced grade padding" is now aligned (it looked odd before)

@brindy brindy self-assigned this Mar 1, 2018
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM

@subsymbolic subsymbolic merged commit e83e2c3 into develop Mar 1, 2018
@subsymbolic subsymbolic deleted the feature/fix_scorecard_padding branch March 1, 2018 11:35
aitorvs pushed a commit that referenced this pull request Nov 18, 2021
* Implement the Report and Issue screen

* Mods due to non-transitive R files

* Review: remove findViewById as we use view binding

* Review: remove commented out code

* Review: remove the refreshTickerChannel and use a better way

* Report an Issue screen - Native form screens [2/n] (#199)

* Implement the Report and Issue form screens

* Fix tests

* Review: call sendAppBreakageReport from the ReportBreakageTextFormActivity instead

* Review: remove the lateinit var adapter

* Review: use Snackbar.LENGTH_LONG instead

* Review: fix copy Where -> Were

* Report an Issue screen - Add VPN metadata to breakage pixel [3/3] (#201)

* Attach vpn metadata to breakage pixel

* FIX: Add scroll view to the breakage single choice screen

* Review: Use NestedScrollView instead of ScrollView

* Review: Fix the letterSpacing for body1 and Body2 styles

* App breakage design review (#204)

* FIX: Add scroll view to the breakage single choice screen

* FIX: Items on Report an issue page are not tappable

* FIX: Make input field optional

* FIX: Incorrect font weight on titles

* FIX: Allow multiple lines in radio button labels

* FIX: The this form link on Report an issue

* Address review comments
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.

2 participants