-
Notifications
You must be signed in to change notification settings - Fork 300
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
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
@praveenojha33 @shreyanshdwivedi @rajvaibhavdubey @AakashMallik please review this |
@shreyanshdwivedi please help why travis is showing error for closing tag saying that it is already closed ..... |
There was a problem hiding this 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the CI.
@praveenojha33 please review |
There was a problem hiding this 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.
@sakshee-19 please open them (by click) |
There was a problem hiding this 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.
@praveenojha33 done |
Please rebase. |
…into fossasia-development
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:
…into development
fixing the terms section alignment Indenting and organizing the code indentation of code Update terms.component.html
@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. |
Fixes #1466
Checklist
master
branch.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 =)