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

Theme Act #231

Open
2 tasks done
HeshamHassan opened this issue Dec 17, 2018 · 33 comments
Open
2 tasks done

Theme Act #231

HeshamHassan opened this issue Dec 17, 2018 · 33 comments
Assignees

Comments

@HeshamHassan
Copy link
Collaborator

HeshamHassan commented Dec 17, 2018

In all pages:

  • Fix the jump that happened to the header when scrolling down and up.
  • Remove the white space under the header.
@mohamedMahdyCc
Copy link
Collaborator

@HeshamHassan , can you please descripe more info about the issue you mean in all cms/pages , or on all website pages ?

@mohamedMahdyCc
Copy link
Collaborator

@HeshamHassan Remove the white space under the header. Done!

@HeshamHassan
Copy link
Collaborator Author

@mohamedMahdyCc
All website pages

@HeshamHassan , can you please descripe more info about the issue you mean in all cms/pages , or on all website pages ?

@mohamedMahdyCc
Copy link
Collaborator

@ahmedelbougha @HeshamHassan
about the header jump when scroll down, this behavior normal and already exist with the original theme source code, kindly you can download new version from it if you want and we can integrate

about page gabs Done! please confirm and close

@HeshamHassan
Copy link
Collaborator Author

  • Remove the white space under the header : Done

@HeshamHassan
Copy link
Collaborator Author

  • Fix the jump that happened to the header when scrolling down and up.

@Ali-Galal : Kindly advice

@mohamedMahdyCc
Copy link
Collaborator

@HeshamHassan
As user when scroll down i need to keep see the page title or course title

@HeshamHassan
Copy link
Collaborator Author

HeshamHassan commented Jan 14, 2019

@mohamedMahdyCc
Need estimation for:

  • Decrease the main image Hight in the home page to fit the screen on launch.

@mohamedMahdyCc
Copy link
Collaborator

@mohamedMahdyCc
Need estimation for:

  • Decrease the main image Hight in the home page to fit the screen on launch.

i suggest to make the page title or course title sticky like the header so you will keep see the titles while scrolling down

Estimation: 2 hrs

@HeshamHassan @Ali-Galal
please confirm if this accepted to start

@Ali-Galal
Copy link
Collaborator

@mohamedMahdyCc
Need estimation for:

  • Decrease the main image Hight in the home page to fit the screen on launch.

i suggest to make the page title or course title sticky like the header so you will keep see the titles while scrolling down

Estimation: 2 hrs

@HeshamHassan @Ali-Galal
please confirm if this accepted to start

@mohamedMahdyCc this is not what we need, I sent the clarification on the whatsapp group

@mohamedMahdyCc
Copy link
Collaborator

@mohamedMahdyCc
Need estimation for:

  • Decrease the main image Hight in the home page to fit the screen on launch.

i suggest to make the page title or course title sticky like the header so you will keep see the titles while scrolling down
Estimation: 2 hrs
@HeshamHassan @Ali-Galal
please confirm if this accepted to start

@mohamedMahdyCc this is not what we need, I sent the clarification on the whatsapp group

@Ali-Galal thanks for info

Estimation: 5hrs

@Ali-Galal
Copy link
Collaborator

@ahmedelbougha I just want to decrease the image hight, the istemation is a little bit high

@Ali-Galal
Copy link
Collaborator

@ahmedelbougha reminder

@Ali-Galal
Copy link
Collaborator

2nd reminder

@mohamedMahdyCc
Copy link
Collaborator

@Ali-Galal Estimation: 5hrs

@Ali-Galal
Copy link
Collaborator

@mohamedMahdyCc approved

@Mahmoud-Ibrahim1
Copy link

@Ali-Galal @mohamedMahdyCc Done! please confirm and close

@Ali-Galal
Copy link
Collaborator

The Act is ok, but the image hight still the same

@MoatazAbdAlmageed
Copy link

@Ali-Galal Please try to clear the cache (or incognito mode ), If the problem still exists please send me a screenshot. @mohamedMahdyCc

@Ali-Galal
Copy link
Collaborator

@MoatazAbdAlmageed I did but still the same
image

@MoatazAbdAlmageed
Copy link

@Ali-Galal could you please provide an image like this
image
I think now image is reduced

@Ali-Galal
Copy link
Collaborator

image

@MoatazAbdAlmageed
Copy link

MoatazAbdAlmageed commented Feb 7, 2019 via email

@Ali-Galal
Copy link
Collaborator

image

@MoatazAbdAlmageed
Copy link

@Ali-Galal Okay I will check but I'm wondering it works in my machine I will test it in another machine also

@MoatazAbdAlmageed
Copy link

@Ali-Galal pushed I will notify you after deploy to the server.

@MoatazAbdAlmageed
Copy link

@Ali-Galal Please check

@Ali-Galal
Copy link
Collaborator

@MoatazAbdAlmageed still the same

@MoatazAbdAlmageed
Copy link

@Ali-Galal I'm using css media quires to make it fit height so could you please provide window.innerWidth by opening the console and write this window.innerWidth

@Ali-Galal
Copy link
Collaborator

image

@MoatazAbdAlmageed
Copy link

@Ali-Galal I mean chrome console like this (just click f12)
image

@Ali-Galal
Copy link
Collaborator

811

@MoatazAbdAlmageed
Copy link

MoatazAbdAlmageed commented Feb 13, 2019 via email

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

No branches or pull requests

6 participants