mirrors
/
hometown
mirror of https://github.com/hometown-fork/hometown.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
hometown/app/views/stream_entries/_status.html.haml

33 lines
1.5 KiB
Plaintext
Raw Normal View History

Improving borders between predecessor/successor statuses
2016-03-21 11:11:27 +00:00
- include_threads ||= false
- is_predecessor ||= false
- is_successor ||= false
significant improvement in microformats markup (#1063) * significant improvement in microformats markup This is a huge improvement and I believe will close #965. Had these microformats reviewed by others in the community to help ensure they are at least correct, if not complete. I did not want to change the structure of the page, and so there it does not fully mark up the entire ancestry chain, or reply chain, only the direct decendants and direct ancestors are correctly associated, but this is likely fine as the most important bit is to have access to the urls for those toots which are now correctly fetchable. * improve code climate * trying to pass code climate tests * code climate * fix p-summary for content warning posts * fix error introduced when merging via github
2017-04-12 19:04:33 +01:00
- direct_reply_id ||= false
- parent_id ||= false
- is_direct_parent = direct_reply_id == status.id
- is_direct_child = parent_id == status.in_reply_to_id
- parent_id ||= false
Adjusting public display of statuses to look similar to logged-in UI, fix #361 with rich OEmbed display via iframe, fix #237 by hiding sensitive content behind a spoiler on public pages
2016-12-18 18:47:11 +00:00
- centered ||= include_threads && !is_predecessor && !is_successor
significant improvement in microformats markup (#1063) * significant improvement in microformats markup This is a huge improvement and I believe will close #965. Had these microformats reviewed by others in the community to help ensure they are at least correct, if not complete. I did not want to change the structure of the page, and so there it does not fully mark up the entire ancestry chain, or reply chain, only the direct decendants and direct ancestors are correctly associated, but this is likely fine as the most important bit is to have access to the urls for those toots which are now correctly fetchable. * improve code climate * trying to pass code climate tests * code climate * fix p-summary for content warning posts * fix error introduced when merging via github
2017-04-12 19:04:33 +01:00
- h_class = microformats_h_class(status, is_predecessor, is_successor, include_threads)
- style_classes = style_classes(status, is_predecessor, is_successor, include_threads)
- mf_classes = microformats_classes(status, is_direct_parent, is_direct_child)
- entry_classes = h_class + ' ' + mf_classes + ' ' + style_classes
Improving borders between predecessor/successor statuses
2016-03-21 11:11:27 +00:00
Clean up h-card mess of divs
2017-01-06 19:24:51 +00:00
- if status.reply? && include_threads
significant improvement in microformats markup (#1063) * significant improvement in microformats markup This is a huge improvement and I believe will close #965. Had these microformats reviewed by others in the community to help ensure they are at least correct, if not complete. I did not want to change the structure of the page, and so there it does not fully mark up the entire ancestry chain, or reply chain, only the direct decendants and direct ancestors are correctly associated, but this is likely fine as the most important bit is to have access to the urls for those toots which are now correctly fetchable. * improve code climate * trying to pass code climate tests * code climate * fix p-summary for content warning posts * fix error introduced when merging via github
2017-04-12 19:04:33 +01:00
= render partial: 'stream_entries/status', collection: @ancestors, as: :status, locals: { is_predecessor: true, direct_reply_id: status.in_reply_to_id}
.entry{ class: entry_classes }
Improving status display design, including replies and threads
2016-02-28 13:02:53 +00:00
Clean up h-card mess of divs
2017-01-06 19:24:51 +00:00
- if status.reblog?
.pre-header
%div.pre-header__icon
= fa_icon('retweet fw')
%span
= link_to TagManager.instance.url_for(status.account), class: 'status__display-name muted' do
Fix issue with missing emojify class in views (#1455) * Add missing emojify class to landing strip * Add missing emojify class to simple_status partial
2017-04-10 20:27:52 +01:00
%strong.emojify= display_name(status.account)
Clean up h-card mess of divs
2017-01-06 19:24:51 +00:00
= t('stream_entries.reblogged')
Refactoring Grape API methods into normal controllers & other things
2016-02-29 18:42:08 +00:00
DRY up reblog vs original status check Checking reblog vs original status was happening in multiple places across the app. For views, this logic was encapsulated in a helper method named `proper_status` but in the other layers of the app, the logic was duplicated. Because the logic is used at all layers of the app, we extracted it into a `Status#proper` method on the model and changed all uses of the logic to use this method. There is now a single source of truth for this condition. We added test coverage to untested methods that got refactored.
2017-04-07 19:18:30 +01:00
= render partial: centered ? 'stream_entries/detailed_status' : 'stream_entries/simple_status', locals: { status: status.proper }
Improving production logs, removing n+1 on media attachments in atom, adding attachments display to static views
2016-09-08 00:24:26 +01:00
Clean up h-card mess of divs
2017-01-06 19:24:51 +00:00
- if include_threads
significant improvement in microformats markup (#1063) * significant improvement in microformats markup This is a huge improvement and I believe will close #965. Had these microformats reviewed by others in the community to help ensure they are at least correct, if not complete. I did not want to change the structure of the page, and so there it does not fully mark up the entire ancestry chain, or reply chain, only the direct decendants and direct ancestors are correctly associated, but this is likely fine as the most important bit is to have access to the urls for those toots which are now correctly fetchable. * improve code climate * trying to pass code climate tests * code climate * fix p-summary for content warning posts * fix error introduced when merging via github
2017-04-12 19:04:33 +01:00
= render partial: 'stream_entries/status', collection: @descendants, as: :status, locals: { is_successor: true, parent_id: status.id}