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

fix: fixing weird vertical shift #3645

Merged
merged 8 commits into from
Dec 22, 2019
Merged

fix: fixing weird vertical shift #3645

merged 8 commits into from
Dec 22, 2019

Conversation

maze-runnar
Copy link
Contributor

@maze-runnar maze-runnar commented Nov 20, 2019

Fixes #3592

Short description of what this resolves:

ezgif com-gif-maker (4)

Checklist

  • I have read the Contribution & Best practices Guide.
  • My branch is up-to-date with the Upstream development branch.
  • The acceptance, integration, unit tests and linter pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@auto-label auto-label bot added fix labels Nov 20, 2019
@maze-runnar
Copy link
Contributor Author

@kushthedude last time i didn't used length this time i am using "gt" helper is {{unverifed-user-message.length}} is greater than zero then the div will show up .

@maze-runnar
Copy link
Contributor Author

please check it . As if i am also wrong this time too even i am thinking it is working 😄

@@ -1,5 +1,4 @@
{{#if (or (eq session.currentRouteName 'events.view.index') (eq session.currentRouteName 'events.view.export'))}}
{{#if (eq model.state 'draft')}}
Copy link
Member

Choose a reason for hiding this comment

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

Revert this

Copy link
Contributor Author

@maze-runnar maze-runnar Nov 25, 2019

Choose a reason for hiding this comment

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

Revert this

I don't get it . please explain a bit.

Copy link
Member

Choose a reason for hiding this comment

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

{{#if (eq model.state 'draft')}}
Let it be on next line only

@kushthedude
Copy link
Member

Upload a GIF showing the transition with latest code commit.

@maze-runnar
Copy link
Contributor Author

maze-runnar commented Dec 5, 2019

@kushthedude it's working
ezgif com-gif-maker

@maze-runnar
Copy link
Contributor Author

@kushthedude @uds5501 please review .

@iamareebjamal iamareebjamal merged commit 41335dd into fossasia:development Dec 22, 2019
@maze-runnar maze-runnar deleted the patch-6 branch January 12, 2020 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Event Dashboard: Weird vertical displacement on Unpublishing Event
3 participants