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.

Sign up for GitHub

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

Jump to bottom

Issue #3 Fix #8

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Issue #3 Fix #8

wants to merge 4 commits into from

Commits on Sep 27, 2016

  1. Restrict icon sizes. 

    Restricted icon sizes to:
- 18×18
- 24x24
- 36×36
- 48×48

If the user requests another size, an icon with the default size is
returned.
    @avazquezpr
    avazquezpr authored Sep 27, 2016
    Configuration menu
    Copy the full SHA
    eed4200 View commit details
    Browse the repository at this point in the history

  2. Restrict stepper. 

    Stepper increments by available values.
    @avazquezpr
    avazquezpr authored Sep 27, 2016
    Configuration menu
    Copy the full SHA
    606d3fb View commit details
    Browse the repository at this point in the history

  3. FORMAT. 

    Removed warnings and added a blank line at the and (per guidelines).
    @avazquezpr
    avazquezpr authored Sep 27, 2016
    Configuration menu
    Copy the full SHA
    f648261 View commit details
    Browse the repository at this point in the history

  4. Changed testIconsAreTheCorrectSize(). 

    If the user request a size other than 18x18, 24x24, 36x36 or 48x48 an icon with the default size will be returned.
    @avazquezpr
    avazquezpr authored Sep 27, 2016
    Configuration menu
    Copy the full SHA
    a61eaaf View commit details
    Browse the repository at this point in the history