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

Dark Theme for dialog box. #99

Merged
merged 4 commits into from
Feb 18, 2015
Merged

Dark Theme for dialog box. #99

merged 4 commits into from
Feb 18, 2015

Conversation

ANURAGX
Copy link
Contributor

@ANURAGX ANURAGX commented Feb 8, 2015

New Dark Theme.

Demo activity is changed in order to switch from dark theme to light theme.

screenshot_2015-02-09-00-01-42
screenshot_2015-02-09-00-01-53
screenshot_2015-02-09-00-02-01

anurag added 2 commits February 8, 2015 23:43
Demo activity is changed in order to switch from dark theme to light theme.
Dialog theme based on declared theme in the manifest
@davidvavra
Copy link
Contributor

Hi,
I see that you updated the code. Please add a comment when you do, I receive notifications only about comments.

The code looks much better now.
Issues:

  • it still not formatted to our coding style. I noticed that the ifs don't have required number of spaces and sometimes there are no {} brackets
  • I don't see the useDarkTheme() method in the builder we discussed
  • Why are you changing ListDialogFragment? I feel that it might be done in cleaner way.

Code reformatted.

Modifying ListDialogFragment to change the list view color when dark theme is set.
And this class is simplified to some extend.
@ANURAGX
Copy link
Contributor Author

ANURAGX commented Feb 17, 2015

I have formatted the code.

Implemented the useDarkTheme method.

And simplified the listDialogFragment class to some extend.

@ANURAGX ANURAGX closed this Feb 17, 2015
@ANURAGX ANURAGX reopened this Feb 17, 2015
@davidvavra
Copy link
Contributor

Almost there :)

It looks like you didn't use our code formatting configuration. Can you reformat with our configuration?

@ANURAGX
Copy link
Contributor Author

ANURAGX commented Feb 17, 2015

I have reformatted the code.

@davidvavra
Copy link
Contributor

It looks like we had some files with bad formatting ourselves :) Thanks for fixing that. Good job and thanks for your contribution!

davidvavra added a commit that referenced this pull request Feb 18, 2015
Support for dark theme (closes #90)
@davidvavra davidvavra merged commit adfbaee into avast:master Feb 18, 2015
@ANURAGX
Copy link
Contributor Author

ANURAGX commented Feb 18, 2015

Its always honour to works for open source project and in return receiving thanks..

Looking forward to work more.

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