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

Adding Mobile Optimizations for the Educator-facing pages. #4027

Merged
merged 59 commits into from May 30, 2018

Conversation

rja907
Copy link
Contributor

@rja907 rja907 commented Mar 20, 2018

Addresses issue #3871

Changes proposed in this pull request:

  • I have added mobile optimizations for the faq page in this commit. I will be following this Pull Request with many more commits.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

faq

Has this branch been QA'd on staging? no

Have you updated the docs? no

Have the tests been updated? no

Reviewer: @ddmck

ddmck and others added 30 commits January 2, 2018 23:42
Update seeds and better error pages.
Unit Filtering and Better Dropdown CSS
Fix activity reports styling issues
I changed the copy under Partner Schools and included a link to our financial aid article.
Updating confusing copy about financial aid
Mostly just fixing Clever school importing.
Deploy Knowledge Center and Subscriptions 2018
Fix string param being compared to integer
@CLAassistant
Copy link

CLAassistant commented Mar 20, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ ddmck
✅ rja907
❌ Amr Thameen


Amr Thameen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

Here, I added some basic styling when the teacher has not assigned any lessons.

Following screenshot showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

empty classes

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

In this commit, I have added mobile optimization for the map page.

According to me, the page could have used some amount of width and height changes.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

map

@codecov-io
Copy link

codecov-io commented Mar 20, 2018

Codecov Report

❗ No coverage uploaded for pull request base (develop@8ae00df). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #4027   +/-   ##
==========================================
  Coverage           ?   34.48%           
==========================================
  Files              ?      820           
  Lines              ?    19407           
  Branches           ?     1734           
==========================================
  Hits               ?     6692           
  Misses             ?    12612           
  Partials           ?      103
Flag Coverage Δ
#jest 20.46% <ø> (?)
#rspec 46.54% <ø> (?)

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 8ae00df...23f4e11. Read the comment docs.

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

In this commit, I have added mobile optimization for the /subscriptions page.

According to me, there was a need to specify the margins and make sure that everything looks neat and clean.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

subscriptions

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

In this commit, I have added mobile optimization for the /teacher/classroom/new page.

The UI was unordered and the padding was not showing up correctly.

new classroom

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

Here, the iframe which was being used was not optimized for mobile view. Also, it had incorrect width and height settings which left a huge area blank on the site.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity for the mobile view.

stats

Also, I made changes for the desktop view. Firstly, the live site is shown and then, we can see the local site that I made changes on.

stats_desktop

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

In this commit, I added some margin so that the pictures look centralized and don't look like they are fixed to the right side of the page.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

mission

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

In this commit, I added a width percentage property which helps in giving a nice scroll effect for the user.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

press

@rja907
Copy link
Contributor Author

rja907 commented Mar 20, 2018

On the /partners page, I saw that the complete iframe was not visible and height changes needed to be made in order to view all the partners.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

partners

@rja907
Copy link
Contributor Author

rja907 commented Mar 21, 2018

I saw the error that is taking place and I think it can be dealt with at a later stage.
It is listed here: npm/npm#18042.

I'll get back to adding optimizations now which are mainly just stylesheets.

@rja907
Copy link
Contributor Author

rja907 commented Mar 22, 2018

In this commit, I saw that alignment was not correct and that margins were not presenting a good UI.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

manage classes

@rja907
Copy link
Contributor Author

rja907 commented Mar 22, 2018

In this commit, I added a mobile friendly UI for the My Account page using CSS Grid. The page was broken and not limiting to the screen width.

Following .gif file showcases the changes. Left side is the local server where I have made changes and the right side is the live site. You can refer to the URLs for authenticity.

my account

@ddmck ddmck merged commit 23f4e11 into empirical-org:develop May 30, 2018
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.

None yet

4 participants