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

update Ticket Section #87

Open
wants to merge 4 commits into
base: development
Choose a base branch
from
Open

Conversation

IHackPy
Copy link

@IHackPy IHackPy commented Nov 7, 2017

I try to fix css of ticket section but when I setup website locally ticket section was not showed on local server so please review . check this issue "Ticket section need improvement #76 "

@IHackPy
Copy link
Author

IHackPy commented Nov 7, 2017

when I inspect code this works fine

@SaptakS
Copy link
Member

SaptakS commented Nov 7, 2017

This solves the problem of text overflow but it creates a problem that the height now are different for different tickets. Please look into that.

@IHackPy
Copy link
Author

IHackPy commented Nov 7, 2017

yeah thanks for reviewing will fix this soon

@SaptakS
Copy link
Member

SaptakS commented Nov 7, 2017

To test in your local system, create a superuser and then login to django admin to create tickets.

@SaptakS
Copy link
Member

SaptakS commented Nov 10, 2017

@himanshuawasthi95 what is the status of this PR?

@IHackPy
Copy link
Author

IHackPy commented Nov 10, 2017 via email

@IHackPy
Copy link
Author

IHackPy commented Nov 12, 2017

hey @SaptakS after create as a superuser still select the ticket to buy portion not visible
pyconp

@SaptakS
Copy link
Member

SaptakS commented Nov 12, 2017

@himanshuawasthi95 you have to login with django admin and create tickets for it to be visible.

@IHackPy
Copy link
Author

IHackPy commented Nov 14, 2017

I removing padding .,mobile view still not good ! I really ashamed on me , CSS blow my head completely

@SaptakS
Copy link
Member

SaptakS commented Nov 14, 2017

No worries. I will check and fix it. 😄

@IHackPy
Copy link
Author

IHackPy commented Nov 14, 2017

once review last change .... in local system look good

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.

2 participants