This is a request from one of our early adopter Forems. We should only
render the user count that appears on most of our sign up CTAs when a
Forem has a large userbase (initially, I'm setting that to 1000 users).
The change also removes a lot of duplicated markup by using some
partials to render the copy in the various auth CTAs.
Hopefully, this is written in such a way that changing opening up more
fine-grained control of these partials is trivial.
* Use APP_NAME rather than APP_DOMAIN in chat channels
APP_NAME may not always be set (it is not "required" per se), which can
cause issues while deriving the pusher_channels. We should use APP_DOMAIN,
which is more common and always has a default value.
* Add TODO to make pusher channels more unique
* Adds ability to publish/unpublish Podcasts via /admin/podacasts/id/edit
- Moves ability to Admin to remove reliability of resource_admin
- Adds published to podcast_params in Admin::PodcastsController
* Adds a test to check the successful publishing of a podcast in podcasts_spec.rb
* Reverts change to podcast.reload in podcasts_spec.rb
* Move to new keyboard shortcuts
* No longer required
* Import order
* Import order
* Add trailing comma
* Remove keyboardShortcutsHandler from exports
* Remove pointless arrow function
* Create keyboardShortcuts.jsx
Add handler for keyboard shortcuts
* Match github's hotkey style
Makes objects clearer.
* Update keyboardShortcuts.jsx
* Allow single character shortcuts
update to allow single character shortcuts/hotkeys
* Add component documentation
Add inline documentation for the component.
* Fix example typo
* Move keyboardShortcuts to a hook
* Fixed useKeyboardShortcuts example
* Rename keyboardShortcuts.jsx to useKeyboardShortcuts.jsx
* Rebuild
* Move isFormField outside render cycle
Move the isFormField function outside of the render cycle.
* Prevent event listener on empty object
Don't add an event listener if the shortcuts property is an empty object or is null
* Add tests for useKeyboardShortcuts
Initial pass for adding tests to useKeyboardShortcuts
* Missed unmount
Missed a whole word...
* Fix typos
* Convert function type
* Add new test cases.
* Missing brackets
* Typo
* Update useKeyboardShortcuts.test.jsx
* Move event listener to document
* Update tests
* Use testing-library/user-event
* Correct fireEvent
* Update keyDown
* Slight tidy
* Change order, just in case
* Swap to render
* Make function async
* Rerun travis
* Add rerender
* Fix render function
* check event.key for fallback
* Bust caches when navigation links are changed
* Remove extra line
* Add admin_action_taken_at to SiteConfig to reference later
* Format timestamp with rfc3339 and fix Specs
Co-authored-by: mstruve <mollylbs@gmail.com>
* Standardizes image placeholder text in SiteConfig
* Adjusts image placeholder text URL per review suggestion
* Pulls image placeholder text out into a constant, IMAGE_PLACEHOLDER
What was wrong?
`initializeCommentDropdown` is called when a new comment is added.
`initializeCommentDropdown` calls `addDropdownListener(dropdown)` on all
`.dropbtn` elements. Elements with the `.dropbtn` class already exist on
the page so an extra listener is added to it whenever a comment is
added.
How was it fixed?
Whenever a dropdown listener is added to an element with the
`.dropbtn` class, an attribute is set on that element so we know it
already has a dropdown listener. When we add listeners to elements with
the `.dropbtn` class in the future, we don't add them to elements with
the attribute.
* Fill organization userpic background fully to avoid empty corners
* Reset `.crayons-logo` ::after pseudo element content
So the barely visible border is hidden for profiles
* Add configurable field for tag minimum feed score
* Use basic feed algorithm for Forems w/ basic
* Don't use cached value for Forems w/ basic feeds
* Revert "Use basic feed algorithm for Forems w/ basic"
This reverts commit f07853c2d5b28592aeae57f9f0313cba5aa6526e.
* Use a configurable hotness score
* Add a discussion comment
* Adjust tag feed min score default to 0
* Revert change to use original 7 score
* Use home feed minimum score
* Update app/controllers/stories_controller.rb
* Update app/controllers/stories_controller.rb
* Update app/controllers/stories_controller.rb
* Update app/controllers/stories_controller.rb
* Update app/controllers/stories_controller.rb
* Update app/controllers/stories_controller.rb
Co-authored-by: Ben Halpern <bendhalpern@gmail.com>
* Fix tag display bug and crayonsify
* Fix some specs
* Remove unused tag js
* Adjust to remove crayons card bg
* Adjust padding
* add spacing below nav
* drop unnecessary classname
* cleanup headers
* links
* Finalize style and add hint to eliminate div jump
Co-authored-by: ludwiczakpawel <ludwiczakpawel@gmail.com>
