Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: No Visual Difference on Clicking Views #1283

Merged
merged 1 commit into from
Mar 20, 2019
Merged

fix: No Visual Difference on Clicking Views #1283

merged 1 commit into from
Mar 20, 2019

Conversation

RotBolt
Copy link

@RotBolt RotBolt commented Mar 12, 2019

Fixes #1281

Changes: [Add here what changes were made in this issue and if possible provide links.]
Add ?selectableItemBackground for making Visual Difference on Clicking them

Screenshots for the change:

@RotBolt RotBolt changed the title Fix No Visual Difference on Clicking Views Fix : No Visual Difference on Clicking Views Mar 12, 2019
@RotBolt RotBolt changed the title Fix : No Visual Difference on Clicking Views fix: No Visual Difference on Clicking Views Mar 12, 2019
@iamareebjamal
Copy link
Member

Change the ripple width to match parent

@RotBolt
Copy link
Author

RotBolt commented Mar 16, 2019

Umm , Actually I intentionally restrained it . As it shows which part of view is clicked .

There's huge blank space on right hand side . Which shouldn't show click effect

@iamareebjamal
Copy link
Member

It should.

@RotBolt
Copy link
Author

RotBolt commented Mar 16, 2019

Ok . Will do it

@RotBolt
Copy link
Author

RotBolt commented Mar 16, 2019

converted to match parent

iamareebjamal
iamareebjamal previously approved these changes Mar 16, 2019
Signed-off-by: thelimitbreaker <rohanmaityofficial@gmail.com>
@RotBolt
Copy link
Author

RotBolt commented Mar 19, 2019

@iamareebjamal I have made requested changes . Please review and if seems proper,.merge it

@iamareebjamal iamareebjamal merged commit 18f258c into fossasia:development Mar 20, 2019
@RotBolt RotBolt deleted the fix/NoVisualDifference branch April 5, 2019 02:36
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