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

Feature/responsive #2891

Merged
merged 2 commits into from
Jan 18, 2018
Merged

Conversation

SaptakS
Copy link
Contributor

@SaptakS SaptakS commented Jan 18, 2018

Status

Ready for review

Description of Changes

This pull request addresses responsiveness of frontend in the headers
and panel sections.
#1357 .

Changes proposed in this pull request:

  • Changed the form division in file uploading page in Source Interface to percentage from pixel
  • The headers and panel section were not aligned properly using float and was breaking in some sizes.
    screen shot 2018-01-18 at 8 19 22 pm

I changed it to use Flexbox instead for media screen width above 768px. Now it looks like this:
screen shot 2018-01-18 at 8 16 34 pm
This fix will affect almost all the pages except for the home page

Checklist

If you made changes to the app code:

  • Unit and functional tests pass on the development VM

Make the width for attachment and message in percentage
instead of pixels.

Signed-off-by: Saptak Sengupta <[email protected]>
Panel selected was wrapping weirdly in some width, so
I changed the code to make it use flex for screen above 768px.
This problem was there in all the websites using panel and
header.

Signed-off-by: Saptak Sengupta <[email protected]>
@SaptakS SaptakS requested a review from a user January 18, 2018 15:37
@codecov-io
Copy link

codecov-io commented Jan 18, 2018

Codecov Report

Merging #2891 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2891   +/-   ##
========================================
  Coverage    85.55%   85.55%           
========================================
  Files           31       31           
  Lines         1917     1917           
  Branches       214      214           
========================================
  Hits          1640     1640           
  Misses         228      228           
  Partials        49       49

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0bd3a97...5fa063f. Read the comment docs.

Copy link
Contributor

@redshiftzero redshiftzero left a comment

Choose a reason for hiding this comment

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

Hi @SaptakS, this looks great! 👍 to merge from me

@redshiftzero redshiftzero merged commit b222515 into freedomofpress:develop Jan 18, 2018
@ghost ghost added feature CSS (Formerly also included SASS) labels Jan 18, 2018
@redshiftzero redshiftzero added this to the 0.6 milestone Feb 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS (Formerly also included SASS) feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants