-
Notifications
You must be signed in to change notification settings - Fork 5
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
Article with Featured " No " has active featuring date #90
Comments
We should move this to the publishing tab and remove the "on show feature" |
Not sure about this. It is an attribute of the article not of the transition. |
It is a publishing option and should not be at this position at all, in my optionen 'featured' should be at the publishing tab too |
And featuring date it self has not much todo if it's on or off |
you are right, I misunderstood your remark |
I suggest another solution. You are right, that featured_up /_ down should be on the publishing tab. BUT the the whole thing then is useless because users will not take notice of this new function. As we must choose between correct and userfriendly |
onshow is already javascript but there is no reason why featured_up/down is handled differently then publish_up/down. Always showing it is the best option and should belong to the publish tab. |
There is a reason why the fields are handled differently: publish_up and publish_down are always valid. Showing the input fields when the swith is "off" is always wrong, no matter on which tab. |
That's not true, publish_up/down required |
Change direction of background gradient for rtl and fix back-to-top button position for rtl
Steps to reproduce the issue
If an article is not featured, the input fields for featuring dates are confusing.
Expected result
Input fields for dates are hidden
Actual result
System information (as much as possible)
Additional comments
The text was updated successfully, but these errors were encountered: