<?xml version="1.0" encoding="utf-8"?>
<xbrl
  xmlns="http://www.xbrl.org/2003/instance"
  xmlns:afg="http://afginc.com/20201028"
  xmlns:dei="http://xbrl.sec.gov/dei/2019-01-31"
  xmlns:link="http://www.xbrl.org/2003/linkbase"
  xmlns:us-gaap="http://fasb.org/us-gaap/2019-01-31"
  xmlns:xbrldi="http://xbrl.org/2006/xbrldi"
  xmlns:xlink="http://www.w3.org/1999/xlink">
    <link:schemaRef xlink:href="afg-20201028.xsd" xlink:type="simple"/>
    <context id="duration_2020-10-28_to_2020-10-28">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_us-gaap-CommonStockMember">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">us-gaap:CommonStockMember</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-SixSubordinatedDebenturesDueNovember152055Member">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">afg:SixSubordinatedDebenturesDueNovember152055Member</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five875SubordinatedDebenturesDueMarch302059Member">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">afg:Five875SubordinatedDebenturesDueMarch302059Member</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five125SubordinatedDebenturesDueDecember152059Member">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">afg:Five125SubordinatedDebenturesDueDecember152059Member</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five625SubordinatedDebenturesDueJune12060Member">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">afg:Five625SubordinatedDebenturesDueJune12060Member</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <context id="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Four5SubordinatedDebenturesDueSeptember152060Member">
        <entity>
            <identifier scheme="http://www.sec.gov/CIK">0001042046</identifier>
            <segment>
                <xbrldi:explicitMember dimension="us-gaap:StatementClassOfStockAxis">afg:Four5SubordinatedDebenturesDueSeptember152060Member</xbrldi:explicitMember>
            </segment>
        </entity>
        <period>
            <startDate>2020-10-28</startDate>
            <endDate>2020-10-28</endDate>
        </period>
    </context>
    <dei:EntityRegistrantName
      contextRef="duration_2020-10-28_to_2020-10-28"
      id="Hidden_dei_EntityRegistrantName">AMERICAN FINANCIAL GROUP INC</dei:EntityRegistrantName>
    <dei:AmendmentFlag contextRef="duration_2020-10-28_to_2020-10-28">false</dei:AmendmentFlag>
    <dei:EntityCentralIndexKey
      contextRef="duration_2020-10-28_to_2020-10-28"
      id="Hidden_dei_EntityCentralIndexKey">0001042046</dei:EntityCentralIndexKey>
    <dei:DocumentType contextRef="duration_2020-10-28_to_2020-10-28">8-K</dei:DocumentType>
    <dei:DocumentPeriodEndDate contextRef="duration_2020-10-28_to_2020-10-28">2020-10-28</dei:DocumentPeriodEndDate>
    <dei:EntityIncorporationStateCountryCode contextRef="duration_2020-10-28_to_2020-10-28">OH</dei:EntityIncorporationStateCountryCode>
    <dei:EntityFileNumber contextRef="duration_2020-10-28_to_2020-10-28">1-13653</dei:EntityFileNumber>
    <dei:EntityTaxIdentificationNumber contextRef="duration_2020-10-28_to_2020-10-28">31-1544320</dei:EntityTaxIdentificationNumber>
    <dei:EntityAddressAddressLine1 contextRef="duration_2020-10-28_to_2020-10-28">301 East Fourth Street</dei:EntityAddressAddressLine1>
    <dei:EntityAddressCityOrTown contextRef="duration_2020-10-28_to_2020-10-28">Cincinnati</dei:EntityAddressCityOrTown>
    <dei:EntityAddressStateOrProvince contextRef="duration_2020-10-28_to_2020-10-28">OH</dei:EntityAddressStateOrProvince>
    <dei:EntityAddressPostalZipCode contextRef="duration_2020-10-28_to_2020-10-28">45202</dei:EntityAddressPostalZipCode>
    <dei:CityAreaCode contextRef="duration_2020-10-28_to_2020-10-28">513</dei:CityAreaCode>
    <dei:LocalPhoneNumber contextRef="duration_2020-10-28_to_2020-10-28">579-2121</dei:LocalPhoneNumber>
    <dei:WrittenCommunications contextRef="duration_2020-10-28_to_2020-10-28">false</dei:WrittenCommunications>
    <dei:SolicitingMaterial contextRef="duration_2020-10-28_to_2020-10-28">false</dei:SolicitingMaterial>
    <dei:PreCommencementTenderOffer contextRef="duration_2020-10-28_to_2020-10-28">false</dei:PreCommencementTenderOffer>
    <dei:PreCommencementIssuerTenderOffer contextRef="duration_2020-10-28_to_2020-10-28">false</dei:PreCommencementIssuerTenderOffer>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_us-gaap-CommonStockMember">Common Stock</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_us-gaap-CommonStockMember">AFG</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_us-gaap-CommonStockMember">NYSE</dei:SecurityExchangeName>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-SixSubordinatedDebenturesDueNovember152055Member">6% Subordinated Debentures due November&#160;15,&#160;2055</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-SixSubordinatedDebenturesDueNovember152055Member">AFGH</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-SixSubordinatedDebenturesDueNovember152055Member">NYSE</dei:SecurityExchangeName>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five875SubordinatedDebenturesDueMarch302059Member">5.875% Subordinated Debentures due March&#160;30,&#160;2059</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five875SubordinatedDebenturesDueMarch302059Member">AFGB</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five875SubordinatedDebenturesDueMarch302059Member">NYSE</dei:SecurityExchangeName>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five125SubordinatedDebenturesDueDecember152059Member">5.125% Subordinated Debentures due December&#160;15,&#160;2059</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five125SubordinatedDebenturesDueDecember152059Member">AFGC</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five125SubordinatedDebenturesDueDecember152059Member">NYSE</dei:SecurityExchangeName>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five625SubordinatedDebenturesDueJune12060Member">5.625% Subordinated Debentures due June&#160;1,&#160;2060</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five625SubordinatedDebenturesDueJune12060Member">AFGD</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Five625SubordinatedDebenturesDueJune12060Member">NYSE</dei:SecurityExchangeName>
    <dei:Security12bTitle contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Four5SubordinatedDebenturesDueSeptember152060Member">4.5% Subordinated Debentures due September&#160;15,&#160;2060</dei:Security12bTitle>
    <dei:TradingSymbol contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Four5SubordinatedDebenturesDueSeptember152060Member">AFGE</dei:TradingSymbol>
    <dei:SecurityExchangeName contextRef="duration_2020-10-28_to_2020-10-28_us-gaap-StatementClassOfStockAxis_afg-Four5SubordinatedDebenturesDueSeptember152060Member">NYSE</dei:SecurityExchangeName>
    <dei:EntityEmergingGrowthCompany contextRef="duration_2020-10-28_to_2020-10-28">false</dei:EntityEmergingGrowthCompany>
</xbrl>
