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

Fixing the terms and privacy (container alignment)(for small screen)(indentation) #1453

Closed
wants to merge 37 commits into from

Conversation

subhahu123
Copy link
Contributor

@subhahu123 subhahu123 commented Feb 3, 2019

Fixes #1466

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream master branch.
  • I have added necessary documentation (if appropriate)
  • Added Surge preview link

Changes proposed in this pull request:

Before

![screen shot 2019-02-03 at 13 16 11](https://user-images.githubusercontent.com/34541684/52174185-84612180-27b6-11e9-8061-f7261c2104e5.png =100x20)
![screen shot 2019-02-03 at 13 18 26](https://user-images.githubusercontent.com/34541684/52174186-84612180-27b6-11e9-97d9-4fd6fd716f7c.png =100x20)

After

![screen shot 2019-02-03 at 13 30 48](https://user-images.githubusercontent.com/34541684/52174320-f84ff980-27b7-11e9-95ef-11f0d1abf65c.png =)
![screen shot 2019-02-03 at 13 30 56](https://user-images.githubusercontent.com/34541684/52174322-f8e89000-27b7-11e9-8112-7811050447a4.png =)

@codecov
Copy link

codecov bot commented Feb 3, 2019

Codecov Report

Merging #1453 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           development   #1453   +/-   ##
===========================================
  Coverage         63.1%   63.1%           
===========================================
  Files               51      51           
  Lines             1450    1450           
  Branches           181     181           
===========================================
  Hits               915     915           
  Misses             425     425           
  Partials           110     110
Impacted Files Coverage Δ
src/app/intelligence/intelligence.component.ts 76% <0%> (ø) ⬆️

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 6212b90...a8a928a. Read the comment docs.

@subhahu123
Copy link
Contributor Author

@praveenojha33 @shreyanshdwivedi @rajvaibhavdubey @AakashMallik please review this

@subhahu123
Copy link
Contributor Author

@shreyanshdwivedi please help why travis is showing error for closing tag


saying that it is already closed .....

@subhahu123 subhahu123 changed the title Fixing the terms and privacy (container alignment) Fixing the terms and privacy (container alignment)(indentation) Feb 3, 2019
Copy link
Contributor

@Pipe-Runner Pipe-Runner left a comment

Choose a reason for hiding this comment

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

Changes look good to me. But CI is failing. Resolve that.

Copy link
Contributor

@sakshee-19 sakshee-19 left a comment

Choose a reason for hiding this comment

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

Fix the CI.

@subhahu123 subhahu123 changed the title Fixing the terms and privacy (container alignment)(indentation) Fixing the terms and privacy (container alignment)(for small screen)(indentation) Feb 13, 2019
@subhahu123
Copy link
Contributor Author

@praveenojha33 please review

Copy link
Contributor

@sakshee-19 sakshee-19 left a comment

Choose a reason for hiding this comment

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

Unable to see the snapshots. Could you please update your PR description.

@subhahu123
Copy link
Contributor Author

Unable to see the snapshots. Could you please update your PR description.

@sakshee-19 please open them (by click)
images was big so i have changed their link to comment section
they are viewable ............

Copy link
Member

@praveenojha33 praveenojha33 left a comment

Choose a reason for hiding this comment

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

Please open an issue and reference this PR.

@subhahu123
Copy link
Contributor Author

Please open an issue and reference this PR.

@praveenojha33 done

@praveenojha33
Copy link
Member

Please rebase.

subhahu123 and others added 14 commits March 1, 2019 11:20
Fixed clear search issue fossasia#1230

Fixed double quotes to single quotes

Fixed reference for navbar html

Fixed spacing issues in constructor scope

Fixed clear search issue fossasia#1230

Add Flex Slider to Help Videos (fossasia#1234)

Switch

Lint
(fossasia#1207)Added link to map address

Analytics border lie inside the window (fossasia#1346)


Refactors Theme Component and added tests for coverage

Fixed Contact Form UI (fossasia#1257)

Fix 1179 - Add SUSI.AI for Magic Mirror, Yaydoc, Meilix and Phimp.me in Projects section under the menus icon in the navbar

Fixed UI of Advanced Search Page

add a share button to infobox

add a share button to infobox

fix indentation

add modal

add icons

fix linting errors

fix icons

fix travis build errors

fix whitespace

fix whitespace

format HTML

add borders

improve styles

increase padding

Style and link fixes on the Privacy and Terms page(fossasia#1261)

Tests Added (fossasia#1278)

* Tests Added

Some of the tests for the files have been added

* Tests Added

Some of the tests for the newly added functions have been added

Fixes fossasia#1327: Contact componnet refactored (fossasia#1328)


Fixes fossasia#1267: Infobox share modal, styling fixes, redundancies removed

fix footer

fixing (fossasia#1351)

Update contact.component.ts
Update contact.component.html
Fixed validations in Contact form

Fixed linting issue

Added share link and made styles changes to infobox-share-popup (fossasia#1349)


Fixes  fossasia#1317: Reducers and Actions cleaned (fossasia#1318)


Fixed formatting of code

Fixed a typo in the contact form

Refactored Help Component and Added tests for increasing coverage

New Design of about component

The design of the about component has been changed

Fixes fossasia#1221

Susper logo Addition
cursor chnages to hover

Update README.md
Fixed minor link and ui issues

depth field only accepts numbers

Loading animation added to result component (fossasia#1204)


Fixes fossasia#1308: Refactor app.modules.ts (fossasia#1309)


Fixing the susper logo alignment and its container
Fixing alignment for all screen sizes
Fixing width of div
Update index.component.html
Update index.component.css
Update index.component.html
Update index.component.css
Update index.component.css
Update index.component.css
Update index.component.css
Fixse fossasia#1306: Removed redundant package delarations (fossasia#1307)


Fixing About page css (fossasia#1298)


Added sidebar menu on Terms page and tests to increase coverage

Fixing the hovering and visited color of links (fossasia#1293)

* Changing color of hovering , Visited link 

Ignoring the overriden of tags of hover and visited

* Changing color

Fix 1268 - Fix overlapping Help Section with footer (fossasia#1271)

* Updated Badgeyay's icon (fossasia#1203)

The badgeyay's icon on susper.com has been updated.

Fixes fossasia#1201

* "Commit "

Added sidebar for Privacy page and tests to increase coverage

Fixes fossasia#1242 corrections in crawl job page (fossasia#1243)


Fixes fossasia#1310: Travis issue fixed

Fix fossasia#1388 : did you mean content is now clickable

Added/Updated tests

Fixes fossasia#1322: About component refactored and cleaned up

Adding rel="noopener" for fixing cross-link links
Update dropdown.css

To hide overflow along x-axis in the dropdown to give UI effect similar to that of Google Search
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Update about.component.html
Adding a lang attribute to the html element
refactored

type changed

Search bar is fixed when scrolled

Removed fixed navbar for mobile

Changed shape of search-bar to curve shape (fossasia#1462)

* Changed shape of search-bar to curve shape

* Improved the corner of search-bar

Merge branch 'development' of https://github.com/fossasia/susper.com into fossasia-development

Merge branch 'fossasia-development' into development

Conflicts:
subhahu123 and others added 4 commits March 14, 2019 22:57
@simsausaurabh
Copy link
Member

@subhahu123 It makes very difficult for us to review PRs with no proper description, un-squashed commits, could you create a separate PR and fix everything. I am closing this one for now.

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.

Bug in terms and privacy component html and css ( for small screens)
5 participants